Skip to content

Conversation

@fscheiner
Copy link
Member

@fscheiner fscheiner commented Nov 17, 2025

  • Enable CI builds for CentOS Stream/Rocky Linux 10
  • Drop --with-ssl-engine from GSI-OpenSSH's configure(.gnu) line
  • Enable builds on arm64 runners in addition

For the not so recently made available ARM64 runners in GHA, see:

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
@fscheiner
Copy link
Member Author

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...

@fscheiner fscheiner requested review from ellert and msalle November 17, 2025 12:36
@fscheiner
Copy link
Member Author

...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.

@fscheiner
Copy link
Member Author

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.

Copy link
Member

@msalle msalle left a 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.

@fscheiner
Copy link
Member Author

I'll try to restart that one failing test, assuming it will work through now...

@fscheiner
Copy link
Member Author

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.

@fscheiner
Copy link
Member Author

fscheiner commented Nov 18, 2025

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

  • Enable CI builds for CentOS Stream/Rocky Linux 10
  • Drop --with-ssl-engine from GSI-OpenSSH's configure(.gnu) line
  • Enable builds on arm64 runners in addition

@fscheiner fscheiner changed the title Enable ci builds on arm64 runners Enable CI builds for *10 distros and also on arm64 runners Nov 18, 2025
@fscheiner
Copy link
Member Author

@msalle : With #240 closed, I though I'd just update the tile and initial comment of this PR accordingly.

@fscheiner fscheiner requested a review from msalle November 18, 2025 08:43
Copy link
Member

@msalle msalle left a 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.

@fscheiner
Copy link
Member Author

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...

@fscheiner fscheiner merged commit 9e70bcf into gridcf:master Nov 18, 2025
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants