-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI: Correct etcd and ncurses package names for Ubuntu 24.04 #16966
Conversation
Signed-off-by: Matt Lord <[email protected]>
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
Might it be prudent to pin to ubuntu-22.04, rather than latest (in particular for the versions older than Vitess 22), so things don't break from under us? |
Signed-off-by: Matt Lord <[email protected]>
Yeah, it looks like we'll probably need to do that. |
42b32f5
to
ece292a
Compare
Or at least, pin to 24.04, so the next upgrade to 26.04 will be explicit (and backport optional). |
This reverts commit ece292a. Signed-off-by: Matt Lord <[email protected]>
Signed-off-by: Matt Lord <[email protected]>
We had everything pinned, but that was changed in #16714. @frouioui Was that done out of necessity there? I know we pinned it explicitly before to avoid upgrade problems. |
Description
In Ubuntu 24.04 — which is currently what is used for
ubuntu-latest
in GitHub Actions — theetcd
package was split intoetcd-client
andetcd-server
and it includes libncurses6.Note
This is not needed today (the driver for opening this seems to have been fixed in Actions) — so there must be package helpers in 24.04 that help bridge the gap and cover the transition, but these may go away in 26.04. Since these are the correct package names, there's no reason not to make the change.
Related Issue(s)
Checklist