Skip to content

Conversation

junaruga
Copy link
Member

Cc: @hsbt

Hello,
I am trying to add GitHub Actions ppc64le/s390x cases to this repository using the services provided by IBM project https://github.com/IBM/actionspz. This PR is working in progress. There are 2 commits. The 1st commit is temporary commit just to remove the unused job for my testing and debugging. I will revert this commit later. The 2nd commit is the main commit.

As a context, Ruby project submitted the request of using this CI by the ticket IBM/actionspz#4, and the ruby/openssl is one of the repositories which we want to add the cases according to @hsbt. So far we applied the cases to the following ruby/* repositories.

I needed to create the new job test-ibm in the .github/workflows/test.yml, because this CI (IBM's self-hosting CI) only works in the registered repositories, in our case, ruby/* repositories. The CI doesn't work in fork repositories. So, I needed to use the if: github.repository == 'ruby/openssl' syntax due to that.

I am using YAML's anchors and aliases feature to prevent duplicated logic in steps between test and test-ibm jobs. The GitHub Actions document is below.

https://docs.github.com/en/actions/reference/workflows-and-actions/reusing-workflow-configurations#yaml-anchors-and-aliases

As I cannot test the GitHub Actions ppc64le/s390x cases in my fork repository. Let me test the cases by sending this PR.

What do you think?

@junaruga junaruga force-pushed the wip/gha-ppc64le-s390x branch from fd3bfde to 1871271 Compare September 24, 2025 19:16
@junaruga junaruga changed the title WIP: CI: Add GitHub Actions ppc64le/s390x cases CI: Add GitHub Actions ppc64le/s390x cases Sep 24, 2025
@junaruga
Copy link
Member Author

I confirmed the ppc64le/s390x cases passed by the CI log below.
https://github.com/ruby/openssl/actions/runs/17986728580/job/51166709117?pr=946

I removed the 1st commit which was used for debugging. Now there is only one commit. I also updated the commit message adding a context.

Copy link
Member

@rhenium rhenium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this!

Changes look good to me. Is this ready to merge?

This commit adds the GitHub Actions ppc64le/s390x cases using the CI services
provided by the project <https://github.com/IBM/actionspz>.

As a context, we submitted the request to use the CI by the ticket
<IBM/actionspz#4>, and the ruby/openssl is one of the
ruby/* repositories that we want to add the CI cases.

This CI doesn't work in fork repositories. Therefore we create the new job
`test-ibm` to use the `if` syntax to run the `test-ibm` job only in ruby/openssl
repository.

I am using YAML's anchors and aliases feature to avoid duplicated logic between
the `test` and `test-ibm` jobs. The document is below.
https://docs.github.com/en/actions/reference/workflows-and-actions/reusing-workflow-configurations#yaml-anchors-and-aliases
@junaruga junaruga force-pushed the wip/gha-ppc64le-s390x branch from 1871271 to b2d7c2a Compare September 25, 2025 16:18
@junaruga junaruga marked this pull request as ready for review September 25, 2025 16:18
@junaruga
Copy link
Member Author

Thanks for your review. Yes, I am ready to merge this PR, and I am going to merge the PR.

@junaruga junaruga merged commit 4b923b4 into ruby:master Sep 25, 2025
43 checks passed
@junaruga junaruga deleted the wip/gha-ppc64le-s390x branch September 25, 2025 16:59
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