-
Notifications
You must be signed in to change notification settings - Fork 29
Enable CI builds for *10 distros and also on arm64 runners #241
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
Enable CI builds for *10 distros and also on arm64 runners #241
Conversation
This is no longer supported with: * RHEL 10 * CentOS Stream 10 * Rocky Linux 10 ...and also not needed anymore when checking the current GSI-OpenSSH sources. Also see gridcf#234.
plus white space cleanup in original workflow file
|
I already tested this locally in my fork of the GCT and it worked for me after some trial and error. So I assume it will also work here, let's see... |
|
...well, unless package data is in an inconsistent state, like for https://github.com/gridcf/gct/actions/runs/19429781076/job/55585691152?pr=241 - or could be an error on GHA side, as the other Rocky Linux 9 build didn't error out so far. |
|
Oh yeah, nearly forgot about this fact: The ARM64 builds are configured do not upload releases packages. This is only done for the regular x86_64 builds in case of the creation of a (release) tag. |
msalle
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.
In general it looks pretty straightforward, especially if you have tested it.
See my one comment though.
|
I'll try to restart that one failing test, assuming it will work through now... |
...and it did this time, so only a temporary error. |
|
Memo: If this PR gets merged instead of #240 and this PR, when making the merge, please combine both titles and descriptions for the merge commit, e.g. like: Enable CI builds for *10 distros and also on arm64 runners
|
msalle
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.
Looks good to me, and adding test and build targets is to me close enough for a single PR. Especially since it simplifies it a bit.
Agreed. I'll merge this now... |
For the not so recently made available ARM64 runners in GHA, see: