-
Notifications
You must be signed in to change notification settings - Fork 2.2k
ci: add checking Go version from Dockerfile #4997
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
Conversation
For some reason, some jobs in .github/workflows/validate.yml have "fetch-depth: 0" argument to actions/checkout, meaning "all history for all branches and tags". Obviously this is not needed here. Signed-off-by: Kir Kolyshkin <[email protected]>
This is to ensure that Go version in Dockerfile (which is used to build release binaries) is: - currently supported; - used in CI tests. Signed-off-by: Kir Kolyshkin <[email protected]>
|
This is a much simpler solution and is enough of a prod for us to go update stuff in the old branches (before we do a release) without auto-updating us to to |
thaJeztah
left a comment
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.
LGTM seems like a sane solution for the problem, thanks!
|
I think the AlmaLinux 8 CI failure is a flake -- I've opened #5003 to track this. This change to our GHA config cannot cause an unrelated error in Cirrus-CI. |
This is to ensure that Go version in Dockerfile (which is used to build
release binaries) is:
Related to #4969 / alternative to #4986.