feat: 22.04 default runner image #2050
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mumoshu
approved these changes
Dec 1, 2022
Collaborator
mumoshu
left a comment
mumoshu
left a comment
There was a problem hiding this comment.
Perfect. Thanks as always for your awesome work
bgrainger
added a commit
to Faithlife/actions-runner-image
that referenced
this pull request
Dec 2, 2022
This synchronizes our image with GitHub hosted runners, which are moving to Ubuntu 22.04: https://github.blog/changelog/2022-11-09-github-actions-ubuntu-latest-workflows-will-use-ubuntu-22-04/. See also the rationale at actions/actions-runner-controller#2050.
Contributor
|
Hi, Any possibility to include back |
|
Hello, same here, I think we are using the summerwind image though, but we're seeing |
|
Removing build-essential has broken a load of our CI/CD workflows that use |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Depends on #2036 #2063
Resolves #2006
To Do
Preample
A new Ubuntu version represents an opportunity for the ARC maintainers to cut down on the installed software in the runner. The existing runner images contain a lot of sofware that would not typically be needed in CI/CD environment or has a niche use case, by including it we get the following negatives:
The software that is included in the runner as it is today was largely inherited and not active choices made by the current maintainers.
Things Removed and Why
build-essential - CI/CD unrelated / niche use case
dnsutils - CI/CD unrelated
ftp - CI/CD unrelated / niche use case
iproute2 - CI/CD unrelated
iputils-ping - CI/CD unrelated
libunwind8 - CI/CD unrelated
locales - CI/CD unrelated
netcat - CI/CD unrelated
openssh-client - CI/CD unrelated / niche use case
parallel - niche use case
rsync - CI/CD unrelated / niche use case
shellcheck - various actions exist to run shellcheck, too opinionated to be in the base runner https://github.com/search?q=shellcheck+action https://github.com/github/super-linter
telnet - CI/CD unrelated
time - CI/CD unrelated
tzdata - CI/CD unrelated
upx - CI/CD unrelated
wget - we already bundle curl which covers the same use case
zstd - CI/CD unrelated
Python
python - there is a official set-up action for Python so no need to bundle it making the base image larger https://github.com/actions/setup-python
python3-pip - python has been removed
When our runner images were first produced GitHub Actions offered very poor support for managing runners and so there was a strong incentive for ARC to have a fatter more inclusive image over a slimmer image. Since those days GitHub has introduced the
--disableupdateflag enabling ARC administrators to more easily use their own runners built from ours without significant operational impacts lessening the incentive to bundle as much as possible in the ARC runners themselves.