Skip to content

Add ARM64 musl coverage across SDK transports - #2548

Open
roji wants to merge 5 commits into
mainfrom
roji-enable-musl-inproc
Open

Add ARM64 musl coverage across SDK transports#2548
roji wants to merge 5 commits into
mainfrom
roji-enable-musl-inproc

Conversation

@roji

@roji roji commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Why

The language SDKs had no CI execution coverage for musl, and Node.js, Python, Go, .NET, and Rust also lacked Linux ARM64 runtime coverage. The Copilot CLI runtime wrapper now supports dynamic musl loading, so these paths can be exercised end to end.

What changed

  • Add a shared Alpine Docker action and ARM64 musl jobs for Node.js, Python, Go, .NET, and Rust.
  • Run both default/out-of-process and in-process transports with the same full test commands used by the existing Linux coverage.
  • Keep .NET's Alpine matrix CAPI-only rather than duplicating backend coverage.
  • Move platform-independent Node.js, Python, and Go validation out of transport matrices so it runs once.
  • Update .NET test helpers to recognize linux-musl-* RIDs.
  • Upgrade Koffi to 3.2.1, which fixes its musl ARM64 native binary used by Node.js in-process hosting.
  • Use dynamic musl linking for Rust tests and isolate Alpine Cargo artifacts from other host architectures.
  • Route changes to the shared Alpine action through the required-check workflow filters.

Java remains unchanged because its build does not yet produce musl classifier artifacts.

Validation

Ran the complete default and in-process Alpine suites natively on ARM64 for Node.js, Python, Go, .NET, and Rust.

roji added 2 commits September 4, 2026 18:46
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 41f8d874-82fa-4398-9863-998a615d6550
Upgrade Koffi for its fixed musl ARM64 native binary and isolate Rust's native container artifacts from other host architectures.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 41f8d874-82fa-4398-9863-998a615d6550
@roji
roji requested a review from a team as a code owner September 6, 2026 09:07
Copilot AI balanced review requested due to automatic review settings September 6, 2026 09:07
Comment thread .github/workflows/python-sdk-tests.yml Fixed
roji added 3 commits September 6, 2026 11:12
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 41f8d874-82fa-4398-9863-998a615d6550
Configure mounted workspaces as trusted Git directories in the shared Alpine action, centralize common Alpine packages, and keep musl detection compatible with the net472 test target.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 41f8d874-82fa-4398-9863-998a615d6550

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

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