Skip to content

fix(ci): do not rely on GitLab.com runner arch variables#3003

Merged
nejch merged 1 commit into
mainfrom
fix/gitlab-os-arch
Sep 30, 2024
Merged

fix(ci): do not rely on GitLab.com runner arch variables#3003
nejch merged 1 commit into
mainfrom
fix/gitlab-os-arch

Conversation

@nejch

@nejch nejch commented Sep 30, 2024

Copy link
Copy Markdown
Member

See https://gitlab.com/python-gitlab/python-gitlab/-/jobs/7943404960 (aarch64 runner), seems like they run their ARM runners with an AMD64 helper binary/host.

Resulted in a failure on multi-arch build =>https://gitlab.com/python-gitlab/python-gitlab/-/jobs/7943404962

Also upstream docs:

The OS/architecture of the GitLab Runner executable. Might not be the same as the environment of the executor.

@JohnVillalovos JohnVillalovos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

Thanks @nejch

@nejch

nejch commented Sep 30, 2024

Copy link
Copy Markdown
Member Author

Thanks for the review @JohnVillalovos 🙇

@nejch nejch merged commit c848d12 into main Sep 30, 2024
@nejch nejch deleted the fix/gitlab-os-arch branch September 30, 2024 17:32
@nejch

nejch commented Sep 30, 2024

Copy link
Copy Markdown
Member Author

I'll try to release this now to see if it works and also to get #2998 back in.

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.

3 participants