images: add ubuntu/26.04#1798
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new Containerfile and an extra-packages list to build an Ubuntu 26.04 base image for Toolbox containers. The review feedback highlights several opportunities to improve build robustness, such as correcting a typo in the metadata labels, using the -f flag with rm to prevent build failures if targeted files are missing, and safely deleting the ubuntu user only if it exists.
|
|
||
| # Remove apt configuration optimized for containers | ||
| # Remove docker-gzip-indexes to help with "command-not-found" | ||
| RUN rm /etc/apt/apt.conf.d/docker-gzip-indexes /etc/apt/apt.conf.d/docker-no-languages |
There was a problem hiding this comment.
Using rm without the -f (force) flag can cause the build to fail if these configuration files are missing or renamed in future versions of the base image. Using rm -f ensures the build is more robust.
RUN rm -f /etc/apt/apt.conf.d/docker-gzip-indexes /etc/apt/apt.conf.d/docker-no-languages
|
|
||
| # Having anything in /home prevents toolbox from symlinking /var/home there, | ||
| # and 'ubuntu' user with UID 1000 will most likely conflict with host user as well | ||
| RUN userdel --remove ubuntu |
There was a problem hiding this comment.
If the ubuntu user does not exist in the base image (or is removed in a future upstream update), userdel will fail with a non-zero exit code, causing the image build to fail. It is safer to check if the user exists before attempting to delete it, or ignore the failure.
RUN id -u ubuntu &>/dev/null && userdel --remove ubuntu || true
| RUN userdel --remove ubuntu | ||
|
|
||
| # Disable APT ESM hook which tries to enable some systemd services on each apt invocation | ||
| RUN rm /etc/apt/apt.conf.d/20apt-esm-hook.conf |
There was a problem hiding this comment.
Using rm without the -f flag will cause the build to fail if the ESM hook file is not present. Using rm -f is safer and more robust against changes in the base image.
RUN rm -f /etc/apt/apt.conf.d/20apt-esm-hook.conf
debarshiray
left a comment
debarshiray
left a comment
There was a problem hiding this comment.
Thanks a lot for adding the image definition for Ubuntu 26.04! I took a quick look at it, and it seems sane to me. I will wait a bit to hear from @Jmennius because he is our Ubuntu maintainer before merging it.
Meanwhile, I left some comments below; and could you please add the URL of this pull request to the commit message, and tweak it to something like this to match the format we use elsewhere:
.github/workflows, images/ubuntu: Add image definition for Ubuntu 26.04
containers#1798 Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
Thanks for the feedback, I have done the changes. PTAL. |
|
recheck |
containers#1798 Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
debarshiray
left a comment
debarshiray
left a comment
There was a problem hiding this comment.
Thanks a lot for updating the pull request, @Ferenc- ! Everything looks great.
Normally, I would wait for @Jmennius to merge, but you look like an enthusiastic first-time contributor and we have been missing the image since Ubuntu 26.04 was released on the 23rd of April, so I will make an exception and merge once the CI finishes running. :)
|
We are missing an image for Ubuntu 25.10. @Ferenc- would you be interested in submitting a separate pull request to add it? |
This is odd: |
|
recheck |
Yes, I could do that too. |
|
And I suppose the |
Yes, indeed, Ubuntu 24.10 reached End of Life on the 10th of July 2025. Let me insert a commit before yours to see if it helps with the CI. |
Ubuntu 24.10 reached End of Life on 10th July 2025: https://endoflife.date/ubuntu containers#1798
containers#1798 Signed-off-by: Ferenc Géczi <ferenc.gm@gmail.com>
I am going to take a leap of faith and ignore this failure on the assumption that it's something to do with Fedora Rawhide or the Software Factory infrastructure, because everything was fine 5 days ago and nothing relevant to these tests changed since then. |
|
Thank you for playing with Toolbx, @Ferenc- ; and congratulations on your first contribution! :) I am looking forward to your pull request for the Ubuntu 25.10 image. |
2 participants
Add Ubuntu 26.04 image.