Skip to content
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

Add aarch64 checks and auth to workflow #180

Merged
merged 5 commits into from
Jun 17, 2024
Merged

Conversation

joinemm
Copy link
Collaborator

@joinemm joinemm commented Jun 11, 2024

  • Added hetzarm as remote builder so aarch64 configs can be built.
  • Added build3 as remote builder to speed up building of the x86 targets.
  • Added authentication steps to the workflow now that we are dealing with secrets and have to use pull_request_target. The auth steps are similar what is being used in ghaf repo but simplified. Reasoning explained here https://ssrc.atlassian.net/wiki/x/OABjMg

FYI the checks for this PR will fail as it's not able to use the secrets yet.

Tests done in my fork using alt account to make different PRs https://github.com/joinemm/ghaf-infra/pulls

Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
@joinemm joinemm force-pushed the pr-arm-checks branch 2 times, most recently from 44e2861 to 6fa602d Compare June 14, 2024 11:17
Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
@joinemm joinemm marked this pull request as ready for review June 14, 2024 11:23
@joinemm joinemm changed the title Add aarch64 remote builder for CI tests Fix aarch64 checks and add auth to workflow Jun 14, 2024
@joinemm joinemm changed the title Fix aarch64 checks and add auth to workflow Add aarch64 checks and auth to workflow Jun 14, 2024
@joinemm joinemm requested a review from a team June 14, 2024 11:38
Signed-off-by: Joonas Rautiola <joonas.rautiola@unikie.com>
Copy link
Collaborator

@henrirosten henrirosten 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, this seems fine but I don't have means to test this right now.
I assume @joinemm that you have been fully testing this in your fork. If that's the case, I'm fine with merging this.

@joinemm joinemm merged commit bb3806a into tiiuae:main Jun 17, 2024
1 check passed
@joinemm joinemm deleted the pr-arm-checks branch June 17, 2024 11:44
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.

None yet

2 participants