Skip to content

[release-branch.go1.27] ci: add Darwin arm64 nocgo coverage - #2630

Merged
George Adams (gdams) merged 1 commit into
microsoft/release-branch.go1.27from
dev/gadams/backport-darwin-arm64-nocgo-go1.27
Oct 22, 2026
Merged

George Adams (gdams) merged 1 commit into
microsoft/release-branch.go1.27from
dev/gadams/backport-darwin-arm64-nocgo-go1.27

Conversation

@gdams

Copy link
Copy Markdown
Member

Backport of #2626 to microsoft/release-branch.go1.27.

Summary

  • Add the missing Darwin arm64 nocgo inner-loop job, matching the existing amd64 check.
  • Clean cherry-pick of f8d6077; system crypto remains enabled by default on Go 1.27.
  • Reuse the existing native ARM64 pool and build/test infrastructure. One matrix line added; no pool, backend, or submodule changes.

Validation

  • go -C eng/_util test -mod=readonly -count=1 -v ./cmd/pipelineymlgen passed.
  • YAML assertions confirmed exactly one native Darwin nocgo builder per architecture with system crypto enabled.
  • Whitespace checks passed.

The complete Darwin arm64 nocgo build/test job will be validated by CI.

Add the missing Darwin arm64 nocgo inner-loop builder alongside the existing amd64 check. Reuse the native arm64 pool and existing build/test configuration with CGO_ENABLED=0 and system crypto enabled.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
(cherry picked from commit f8d6077)
Copilot AI balanced review requested due to automatic review settings October 22, 2026 09:14
@gdams
George Adams (gdams) requested a review from a team as a code owner October 22, 2026 09:14
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

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.

🟢 Approval recommended

The single matrix addition follows established configuration paths with no blocking issues identified; full build/test execution remains for CI.

No open issues 🎉

What changed in this PR

Backports Darwin arm64 nocgo CI coverage to the Go 1.27 release branch, matching the existing amd64 check.

Changes:

  • Adds one Darwin arm64 nocgo inner-loop job with system crypto enabled by default.
  • Reuses the existing native ARM64 pool and build/test infrastructure without backend or submodule changes.
File Description
eng/​pipeline/​stages/​go-builder-matrix-stages.yml Adds the Darwin arm64 nocgo matrix entry.

Review effort: Balanced

@gdams
George Adams (gdams) merged commit cc47d75 into microsoft/release-branch.go1.27 Oct 22, 2026
60 checks passed
@gdams
George Adams (gdams) deleted the dev/gadams/backport-darwin-arm64-nocgo-go1.27 branch October 22, 2026 10:31
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