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

feat: support linux musl #454

Merged
merged 13 commits into from
Jan 23, 2025
Merged

Conversation

YOU54F
Copy link
Member

@YOU54F YOU54F commented Sep 5, 2024

fixes #453

having some fairly consistent segfaults exclusively on alpine that need investigating further during provider verifications.

consumer side seems fine.

This PR will enable support for Linux Alpine users.

Support compatibility has been added to the matrix, with caveats

⚠️ - Known issues

  • Linux (musl) provider verification is known to experience segmentation faults, reproducible in our CI system.
    • Assistance is greatly appreciated if musl support is important to you.

A workflow dispatch action has been created, and all provider verification tests (those tagged with provider) are not tested on alpine. The workflow can be run manually with SKIP_ALPINE_PROVIDER_TESTS value set to false which is the default for wf dispatch jobs.

This will allow end users to at least utilise pact-go with alpine in some capacity and may lead to an inquisitive user working out why we are getting segfaults in the verifier execution

It also allows us as maintainers to quickly test with any new changes, if the issue is solved, by creating reproducible conditions.

@YOU54F YOU54F marked this pull request as ready for review January 22, 2025 18:41
@coveralls
Copy link

Coverage Status

coverage: 28.209% (-1.0%) from 29.225%
when pulling 3d48a9d on YOU54F:feat/linux-musl
into 0de92f4 on pact-foundation:master.

@YOU54F YOU54F requested a review from mefellows January 22, 2025 18:42
@YOU54F YOU54F merged commit c74d37a into pact-foundation:master Jan 23, 2025
19 checks passed
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.

RFC: Support Alpine Linux
2 participants