Skip to content

[release/10.0] Backport renewable Azure CLI authentication for Helix - #17642

Merged
missymessa merged 2 commits into
dotnet:release/10.0from
missymessa:backport/17592-release-10.0
Oct 7, 2026
Merged

missymessa merged 2 commits into
dotnet:release/10.0from
missymessa:backport/17592-release-10.0

Conversation

@missymessa

Copy link
Copy Markdown
Member

Backport

Backport of #17592 to release/10.0.

Summary

  • Run Entra-authenticated Helix submission and monitoring inside AzureCLI@2 with keepAzSessionActive: true.
  • Make the Helix SDK and standalone Job Monitor prefer AzureCliCredential while retaining existing credential fallbacks.
  • Preserve release/10.0 support for HelixEntraScope.
  • Preserve existing PAT and anonymous behavior.

Validation

  • Microsoft.DotNet.Helix.Sdk.Tests passed with 0 warnings and 0 errors on release/10.0.
  • eng/common/run-helix-job-monitor.sh passed Bash syntax validation.

Tracks AB#12269.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: b83ddd94-a1da-4c3c-aead-1d1643a6b83f
Copilot-Session: 40ffbc91-7138-4298-b61c-73a788185f11
Copilot-Session: 3ee73e53-8098-4533-b3f9-1686640d4f55
Copilot-Session: 0a4ed861-2664-48de-a412-84d7adccd0f0
Copilot-Session: 1672b4d9-b4c1-4736-83fa-c8b06a6f966b
# Conflicts:
#	Documentation/AzureDevOps/SendingJobsToHelix.md
#	eng/common/core-templates/job/helix-job-monitor.yml
#	eng/common/core-templates/steps/send-to-helix.yml
#	src/Microsoft.DotNet.Helix/JobMonitor/JobMonitorRunner.cs
#	src/Microsoft.DotNet.Helix/Sdk/HelixTask.cs
#	src/Microsoft.DotNet.Helix/Sdk/Readme.md
Copilot AI balanced review requested due to automatic review settings September 30, 2026 22:28

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 review overview

🟢 Approval recommended

The backport consistently preserves existing authentication paths while applying the validated renewable Azure CLI flow.

Review effort: Balanced
Findings: None

What changed in this PR

Backports renewable Azure CLI authentication for long-running Helix submissions and monitoring on release/10.0.

Changes:

  • Runs Entra-authenticated Helix operations inside renewable AzureCLI@2 sessions.
  • Prioritizes AzureCliCredential for Helix while preserving fallback, PAT, anonymous, and custom-scope behavior.
  • Extracts shared monitor execution into a reusable shell script and updates documentation.
File Description
Documentation/​AzureDevOps/​SendingJobsToHelix.md Documents renewable WIF authentication.
src/​Microsoft.DotNet.Helix/​Sdk/​Readme.md Updates monitor authentication guidance.
src/​Microsoft.DotNet.Helix/​Sdk/​HelixTask.cs Prefers Azure CLI credentials for submission.
src/​Microsoft.DotNet.Helix/​JobMonitor/​JobMonitorRunner.cs Prefers Azure CLI credentials for monitoring.
src/​Microsoft.DotNet.ArcadeAzureIntegration/​DefaultIdentityTokenCredentialOptions.cs Adds the CLI-preference option.
src/​Microsoft.DotNet.ArcadeAzureIntegration/​DefaultIdentityTokenCredential.cs Implements configurable credential ordering.
eng/​common/​run-helix-job-monitor.sh Centralizes monitor invocation.
eng/​common/​core-templates/​steps/​send-to-helix.yml Runs authenticated submissions in Azure CLI tasks.
eng/​common/​core-templates/​job/​helix-job-monitor.yml Runs authenticated monitoring in a renewable session.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI balanced review requested due to automatic review settings October 1, 2026 16:44
@missymessa
missymessa marked this pull request as ready for review October 1, 2026 16:44
@missymessa
missymessa requested a review from mmitche October 1, 2026 16:44
@missymessa
missymessa enabled auto-merge (squash) October 1, 2026 16:44

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 review overview

🟢 Approval recommended

The backport preserves release-specific scope support, matches the validated upstream design, and introduces no unresolved issues.

Review effort: Balanced
Findings: None

@missymessa
missymessa merged commit de95c64 into dotnet:release/10.0 Oct 7, 2026
17 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.

3 participants