Skip to content
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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davidscholberg
Copy link

@davidscholberg davidscholberg commented Oct 17, 2024

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:

diff -urN images/ubuntu/24.04/Containerfile images/ubuntu/24.10/Containerfile
--- images/ubuntu/24.04/Containerfile   2024-10-16 21:41:33.928967339 -0400
+++ images/ubuntu/24.10/Containerfile   2024-10-16 22:09:08.875131964 -0400
@@ -1,8 +1,8 @@
-FROM docker.io/library/ubuntu:24.04
+FROM docker.io/library/ubuntu:24.10

 LABEL com.github.containers.toolbox="true" \
       name="ubuntu-toolbox" \
-      version="24.04" \
+      version="24.10" \
       usage="This image is meant to be used with the toolbox command" \
       summary="Base image for creating Ubuntu Toolbx containers" \
       maintainer="Ievgen Popovych <[email protected]>"
@@ -21,9 +21,10 @@
 # Install ubuntu-minimal & ubuntu-standard
 # Install extra packages as well as libnss-myhostname
 COPY extra-packages /
-RUN sed -Ei '/apt-get (update|upgrade)/s/^/#/' /usr/local/sbin/unminimize && \
-    apt-get update && \
-    yes | /usr/local/sbin/unminimize && \
+RUN apt-get update && \
+    DEBIAN_FRONTEND=noninteractive apt-get -y install unminimize && \
+    sed -Ei '/apt-get (update|upgrade)/s/^/#/' /usr/bin/unminimize && \
+    yes | /usr/bin/unminimize && \
     DEBIAN_FRONTEND=noninteractive apt-get -y install \
         ubuntu-minimal ubuntu-standard \
         libnss-myhostname \

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.

Copy link

Copy link

@tom-reinders tom-reinders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davidscholberg
Copy link
Author

@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.

Copy link

@Jmennius
Copy link
Collaborator

@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.

So the latest_release should stay with the latest LTS version.

Copy link
Collaborator

@Jmennius Jmennius left a 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)

@davidscholberg
Copy link
Author

@Jmennius updated PR with revert of latest_release back to LTS, and removed 23.10 from that file, and deleted the 23.10 directory as well.

@davidscholberg davidscholberg changed the title Added ubuntu-toolbox version 24.10 Added ubuntu-toolbox version 24.10, removed 23.10 (eol) Oct 18, 2024
Copy link

Comment on lines -14 to -18
# 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
Copy link
Collaborator

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.

@Jmennius Jmennius self-assigned this Oct 20, 2024
@Jmennius
Copy link
Collaborator

@davidscholberg can you add a sign-off to the commit please? (git commit --amend -s, just be sure to fetch the updated branch)

Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants