-
Notifications
You must be signed in to change notification settings - Fork 47
introduce an automation for GHA runner version update #3034
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
base: main
Are you sure you want to change the base?
Conversation
[static] Signed-off-by: Mateusz Błażejewski <[email protected]>
I think you could temporarily remove the schedule and manually trigger the action from the UI to see that it works as expected, no? |
julientinguely-da
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.
Besides doing this test, it looks good to me! Thanks!
[static] Signed-off-by: Mateusz Błażejewski <[email protected]>
6259621 to
b459d1a
Compare
| workflow_dispatch: | ||
|
|
||
| jobs: | ||
| bump_gha_runner_version: |
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.
Do we get notified if this fails? Not 100% sure but I think you need some more boilerplate for that...
S.a.:
| name: Notify Failures |
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.
(Unless it's super annoying to add, let's add notifications; we're very likely to not notice this getting hard broken otherwise.)
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've copied the job from other workflows. I hope it works.
| git add --all | ||
| updated_branch="gha-runner-version-bump-$(date +%Y-%m-%d)" | ||
| git switch -c "${updated_branch}" | ||
| git commit -m "[static] bump GHA runner version to the latest (auto-generated)" -s |
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.
Is "build docker images" part of [static]?
If not we might want to either:
- add that (follow-up PR?)
- do a docker-build here on (only)
splice-test-docker-runner?
I think I like the second option more (we change images ~rarely and when we do we typically also do full cluster tests which do build docker images), assuming that 1. we can do that without forcing a full splice build ( 🤞 ) and 2. we get notifications and a failures board entry in case this flow fails (see my other 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.
Good idea! I've added explicit docker-build for these two images.
Can't you also trigger a GHA run from a branch? We do that for CircleCI runs all the time. But clicking around with your branch the new workflow doesn't seem to show up, sad. @isegall-da do you know of a good way to test this without merging to main first? (Which I'm not against here, for the record, just seems nicer if we can have a feedback loop that doesn't involve merging a PR.) |
I think you need to add |
I was close, it's |
@isegall-da I do have this configured and I'm still not able to trigger it through
So it seems that I need some form of dummy definition for a workflow on |
[static] Signed-off-by: Mateusz Błażejewski <[email protected]>
Argh, that's annoying. Oh, well, seems like "merge to main and test there" is the answer. |
This introduces a new workflow that will fire roughly once every two weeks. It will update the version of the GHA runner base image that is used to build
splice-test-docker-runnerandsplice-test-runner-hookimages.The shell script was tested locally. I'm not sure though if the whole solution is correct but I don't know how to test the actual workflow without creating it.