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

crashpad/tests: run only arm64 tests on macOS, skip x86_64 #65

Merged
merged 1 commit into from
May 30, 2024

Conversation

gm4sl
Copy link
Collaborator

@gm4sl gm4sl commented May 30, 2024

Presently, tests run for macOS/x64_64 hang when run as a gitlab job. As there is still macOS coverage, as arm64, disable the x64_64 tests on macOS, for now, until we have time to properly investigate.

@gm4sl gm4sl requested a review from konraddysput May 30, 2024 00:38
@@ -117,7 +117,7 @@ jobs:
strategy:
fail-fast: false
matrix:
arch: [arm64, x86_64]
Copy link
Collaborator

Choose a reason for hiding this comment

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

There is a check below for running tests on specific architecture. We still want to verify building crashpad on x86.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated so that we continue to build for macOS/x64_64. The tests have been disabled as suggested.

Presently, tests run for macOS hang when run as a gitlab job. Disable the
x64_64 tests on macOS or now (arm64 were already disabled), until we have
time to properly investigate.
@gm4sl gm4sl force-pushed the gmsl/disable_macos_x86_64 branch from c4387af to cc21e4a Compare May 30, 2024 14:29
@gm4sl gm4sl merged commit eeef0df into backtrace May 30, 2024
15 checks passed
@gm4sl gm4sl deleted the gmsl/disable_macos_x86_64 branch May 30, 2024 14:35
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