Skip to content

Add ARM64 support for integration tests in GitHub workflows #6059

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

Merged
merged 3 commits into from
May 20, 2025

Conversation

amartinezfayo
Copy link
Member

@amartinezfayo amartinezfayo commented May 9, 2025

  • Add ARM64 support to integration tests in GitHub workflows.
  • Ensure integration-k8s is included as a required check for success (was previously missing).

Fixes #3740.

@amartinezfayo amartinezfayo force-pushed the arm64-integration-tests branch from a2a0b63 to faac1a3 Compare May 9, 2025 13:32
@amartinezfayo amartinezfayo added this to the 1.12.2 milestone May 9, 2025
@amartinezfayo amartinezfayo force-pushed the arm64-integration-tests branch from faac1a3 to faca7aa Compare May 9, 2025 13:34
@sorindumitru sorindumitru self-assigned this May 13, 2025
integration:
name: integration (linux)
runs-on: ubuntu-22.04
name: integration (${{ matrix.arch }})
Copy link
Collaborator

Choose a reason for hiding this comment

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

One small nit I have for this is that the name of the integration steps have lost the job index:

- PR Build / integration (linux) (5, 4)
+ PR Build / integration (x64)

I wonder if that's something that we can bring back, maybe something like:

Suggested change
name: integration (${{ matrix.arch }})
name: integration (${{ matrix.arch }}) (${{ strategy.job-index}}/${{ strategy.job-total }})

Signed-off-by: Agustín Martínez Fayó <[email protected]>
Copy link
Collaborator

@sorindumitru sorindumitru left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @amartinezfayo!

@amartinezfayo amartinezfayo merged commit 0b0b86a into spiffe:main May 20, 2025
188 of 191 checks passed
tjons pushed a commit to tjons/spire that referenced this pull request May 21, 2025
)

* Add ARM64 support for integration tests on GitHub workflows

Signed-off-by: Agustín Martínez Fayó <[email protected]>
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.

Run integration tests for arm64 on release builds
2 participants