Skip to content

Update dockerfile to ubuntu:24.04 #3771

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

joshua-clayton
Copy link

The Dockerfile has gotten pretty stale, and as a base for build images,
leaves a lot of CVEs and vulnerabilities which are easily fixed by moving to a more recent base.

Update it to the most recent Ubuntu LTS.

I'm not sure if there is a more recent version of the dotnet image, but it seemed simpler to
me to follow the installation instructions for action-runner in the Dockerfile.

Lastly, I added apt-get upgrade to the beginning of each apt command,
which will keep the image up to date with the source image if it is rebuilt,
and apt-get clean to the end of each apt command, which will keep the resultant
layers smaller by removing apt download cache leftovers

In addition to cleaning up the apt lists, remove downloaded packages
This will reduce the size of the layers by a litte bit
Update from Jammy to Noble.
Add the dotnet runtime dependencies to Dockerfile
@joshua-clayton joshua-clayton requested a review from a team as a code owner March 27, 2025 18:14
@joshua-clayton
Copy link
Author

joshua-clayton commented Mar 27, 2025

Also, I don't know if your test suite will cover the Dockerfile, but I can confirm that I am using the above as a base for my self hosted runner. So its building and running with the github api. The only change I had to make to a downstream Dockerimage was to add "--break-system-packages" to the subsequent pip3 command in my local build image

@gthomson31
Copy link

Thanks for this Joshua - hopefully they update it soon

Update github and docker dependencies.
Fill in missing linux dependency version ARGS
@joshua-clayton
Copy link
Author

While waiting for review, I'm updating the dependencies to reflect releases since march. Once this is accepted, the internal github dependencies will be kept up to date by Github's own release process, but while I'm waiting for this to be merged, I need to keep my forked copy up to date.

@joshua-clayton
Copy link
Author

@TingluoHuang I'd love to get this upgrade merged. Any objection to moving to Ubuntu 24.04?

sudo \
unzip \
zlib1g \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/*

# Configure git-core/ppa based on guidance here: https://git-scm.com/download/linux
RUN add-apt-repository ppa:git-core/ppa \
Copy link

@marcispauls marcispauls Jun 4, 2025

Choose a reason for hiding this comment

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

combine together with previous run not to make extra layer + i don't see any reasoning to add extra repo just to get upstream git version

Copy link

@Firas2515 Firas2515 left a comment

Choose a reason for hiding this comment

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

E

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.

4 participants