Skip to content

[DO NOT LAND] Test alpine release failure on CI#8557

Closed
kripken wants to merge 2 commits intoWebAssembly:mainfrom
kripken:focus
Closed

[DO NOT LAND] Test alpine release failure on CI#8557
kripken wants to merge 2 commits intoWebAssembly:mainfrom
kripken:focus

Conversation

@kripken
Copy link
Copy Markdown
Member

@kripken kripken commented Mar 31, 2026

Spec tests fail on alpine during Create-Release. Bisecting that here.

Error:

https://github.com/WebAssembly/binaryen/actions/runs/23809435760/job/69399962319

From creating version_129

https://github.com/WebAssembly/binaryen/actions/runs/23809435760

@kripken kripken marked this pull request as ready for review March 31, 2026 18:18
@kripken kripken requested a review from a team as a code owner March 31, 2026 18:18
@kripken kripken requested review from aheejin and removed request for a team and aheejin March 31, 2026 18:18
@kripken
Copy link
Copy Markdown
Member Author

kripken commented Mar 31, 2026

Sorry, did not mean to request review. I am trying to figure out why github is not running CI here, and thought it might be the "draft" mode. It might also be the merge conflicts, but I need those to test...

@kripken
Copy link
Copy Markdown
Member Author

kripken commented Mar 31, 2026

Not sure I can bisect here, if it won't let me run older commits that conflict with current main... updated to latest, to confirm the error here in CI at least.

@kripken
Copy link
Copy Markdown
Member Author

kripken commented Mar 31, 2026

This confirms the situation: we error on linux+musl+x86_64 - something we only run in Create-Release, normally (to make a portable build) - but do NOT error on the same but with ARM. We also do not error on the same but without musl (i.e. linux+x86_64 on ubuntu, not tested in this PR).

@kripken kripken closed this Mar 31, 2026
@kripken kripken deleted the focus branch March 31, 2026 18:59
kripken added a commit that referenced this pull request Apr 1, 2026
…ng (#8558)

This was noticed during creating the 129 release:
#8557 (comment)

Add an alpine build to CI for x86_64, mirroring our release builds, so
we do not lack coverage (which led to the issue only showing on release, not
CI).

Also make the alpine CI build identical to alpine release (copied from
`create_release`).
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.

1 participant