镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

[release/10.0] Disable SDK certificate generation in macOS Helix - #70542

Merged
wtgodbe merged 1 commit into
dotnet:release/10.0from
wtgodbe:infrastructure/disable-helix-first-run-cert-release10-2026-11-12
Nov 12, 2026
Merged

wtgodbe merged 1 commit into
dotnet:release/10.0from
wtgodbe:infrastructure/disable-helix-first-run-cert-release10-2026-11-12

Conversation

@wtgodbe

@wtgodbe wtgodbe commented Nov 12, 2026

Copy link
Copy Markdown
Member

Skip automatic SDK HTTPS certificate generation in macOS Helix.

Description

Export DOTNET_GENERATE_ASPNET_CERTIFICATE=false for macOS Helix work items before invoking the SDK. Leave other platforms, queue selections, and explicit certificate-generation APIs unchanged.

The investigation of #70521 found SDK commands stalling during first-run HTTPS certificate initialization. The runner then cancels each setup/discovery command after two minutes with exit code 130, before any tests execute. The existing DOTNET_SKIP_FIRST_TIME_EXPERIENCE=1 does not suppress certificate initialization on these SDKs. The same signature also occurs on the release/10.0 baseline.

Representative failed Helix work item.

This avoids unnecessary automatic certificate/keychain operations; it does not identify or fix the underlying native wait.

Validation

  • bash -n eng/helix/content/runtests.sh and git diff --check passed.
  • Eleven stubbed executions of the actual script passed: macOS x64/ARM64 and internal queues, inherited certificate settings, unchanged Linux/container behavior, runner arguments, and exit-code propagation. These checks do not invoke the real SDK or modify a keychain.
  • Earlier investigation verified five SDK startup operations with fresh CLI profiles and this opt-out using SDK 10.0.103 on macOS ARM64.

Draft pending full macOS Helix validation on the affected x64 pool, including HTTPS/certificate tests. Local smoke checks are not a substitute for that CI run.

Avoid automatic HTTPS certificate initialization before macOS Helix SDK setup and discovery commands. Preserve other platforms and explicit certificate generation.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@wtgodbe
wtgodbe marked this pull request as ready for review November 12, 2026 19:24
@wtgodbe
wtgodbe requested a review from a team as a code owner November 12, 2026 19:24
Copilot AI balanced review requested due to automatic review settings November 12, 2026 19:24

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.

🔵 Needs a closer look

Pending macOS x64 Helix validation leaves it unconfirmed whether disabling automatic certificate provisioning resolves the native startup stall while preserving HTTPS/certificate tests.

No open issues 🎉

What changed in this PR

Disables automatic SDK HTTPS certificate generation in macOS Helix work items to avoid first-run certificate/keychain initialization stalls before tests execute.

Changes:

  • Exports DOTNET_GENERATE_ASPNET_CERTIFICATE=false when the Helix queue name contains OSX, before SDK commands run.
  • Leaves other platforms, queue selections, and explicit certificate-generation APIs unchanged.

Full validation on the affected macOS x64 Helix pool, including HTTPS/certificate tests, remains pending.

File Description
eng/​helix/​content/​runtests.sh Adds the macOS-only SDK certificate-generation opt-out.

Review effort: Balanced

@wtgodbe wtgodbe added the tell-mode Indicates a PR which is being merged during tell-mode label Nov 12, 2026
@wtgodbe
wtgodbe merged commit c7ced38 into dotnet:release/10.0 Nov 12, 2026
33 checks passed
@dotnet-policy-service dotnet-policy-service Bot added this to the 10.0.14 milestone Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tell-mode Indicates a PR which is being merged during tell-mode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants