-
Notifications
You must be signed in to change notification settings - Fork 782
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
Run actions workflow jobs on ubuntu-latest #4251
Run actions workflow jobs on ubuntu-latest #4251
Conversation
.github/workflows/release.yml
Outdated
@@ -21,7 +21,7 @@ jobs: | |||
check-permission: | |||
name: Check permission | |||
if: contains(github.ref, 'refs/heads/master') | |||
runs-on: ubuntu-latest | |||
runs-on: ubuntu-22.04 |
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.
There is no reason to change from ubuntu-latest
. Please leave these as-is to stay current.
.github/workflows/security.yml
Outdated
@@ -11,7 +11,7 @@ on: | |||
jobs: | |||
build: | |||
name: Trivy | |||
runs-on: ubuntu-20.04 | |||
runs-on: ubuntu-22.04 |
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.
Better to update to ubuntu-latest
.
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.
I can change this and the release job to ubuntu-latest
. I'm pretty sure this was a conscious decision though at the time that we made back when we upgraded to 20.04 (#2358) to have them all on the same version as the docker image. That doesn't seem necessary though as we really only need the test suite to run on that version right?
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.
Discussed offline. All workflows will run on ubuntu-latest
going forward.
Run security, validate and release jobs on ubuntu-latest