-
Notifications
You must be signed in to change notification settings - Fork 220
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
Added ubuntu-toolbox version 24.10, removed 23.10 (eol) #1565
Conversation
Build failed. ✔️ unit-test SUCCESS in 5m 32s |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably have to add 24.10
to https://github.com/containers/toolbox/blob/main/.github/workflows/ubuntu-images.yaml#L39
61ca0f2
to
7526eed
Compare
@tom-reinders updated PR with 24.10 added to .github/workflows/ubuntu-images.yaml, and I also update the latest_release option in that file to 24.10. |
Build failed. ✔️ unit-test SUCCESS in 5m 29s |
So the latest_release should stay with the latest LTS version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And we also need to drop 23.10 as it is EOL.
This will also make the PR look like a rename of 23.10 files - easier to review :) (thanks for the explicit diff BTW)
7526eed
to
f06e2f7
Compare
@Jmennius updated PR with revert of |
Build failed. ✔️ unit-test SUCCESS in 5m 34s |
f06e2f7
to
86be839
Compare
# Enable myhostname nss plugin for clean hostname resolution without patching | ||
# hosts (at least for sudo), add it right after 'files' entry. We expect that | ||
# this entry is not present yet. Do this early so that package postinst (which | ||
# adds it too late in the order) skips this step | ||
RUN sed -Ei 's/^(hosts:.*)(\<files\>)\s*(.*)/\1\2 myhostname \3/' /etc/nsswitch.conf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed this as libnss-myhostname
now does the actual configuration for us.
@davidscholberg can you add a sign-off to the commit please? ( |
Build failed. ✔️ unit-test SUCCESS in 5m 28s |
Signed-off-by: David Scholberg <recombinant.vector@gmail.com>
86be839
to
839f0f5
Compare
Done. |
Build failed. ✔️ unit-test SUCCESS in 5m 34s |
This PR adds an ubuntu-toolbox image for ubuntu 24.10 and removes version 23.10 as it has reached EOL.
I copied the files from 24.04 and made a few slight changes (obviously updated the version numbers but also had to explicitly install
unminimize
since it doesn't seem to be included with the image anymore). Here's everything that I changed from 24.04:I'm not sure if any of the other steps in the Containerfile are out of date, but the image does work fine with toolbox from my testing of it.