Skip to content

Revert environment: gating on review-trigger.yml - #36005

Merged
PureWeen merged 1 commit into
mainfrom
pureween/fuzzy-goggles
Jun 18, 2026
Merged

Revert environment: gating on review-trigger.yml#36005
PureWeen merged 1 commit into
mainfrom
pureween/fuzzy-goggles

Conversation

@PureWeen

Copy link
Copy Markdown
Member

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Summary

Removes environment: gh-aw-agents from the trigger-review job in review-trigger.yml.

Why

PR #35974 added environment: gh-aw-agents to review-trigger.yml, which changed the GitHub OIDC token subject from repo:dotnet/maui:ref:refs/heads/main to repo:dotnet/maui:environment:gh-aw-agents. The managed identity (Testing-MI-gh2zdo) behind AZDO_TRIGGER_CLIENT_ID has no federated credential for that environment subject, so every /review command fails with:

AADSTS700213: No matching federated identity record found for presented
assertion subject 'repo:dotnet/maui:environment:gh-aw-agents'

The AZDO_TRIGGER_* secrets have been re-added at the repo level. Once the federated credential is updated (requires Azure access to the managed identity), the environment gating can be re-added.

What changed

  • Removed environment: gh-aw-agents from the trigger-review job (1 line)
  • All gh-aw workflows remain gated — only review-trigger.yml (regular GHA) is affected

The OIDC federated credential on the Testing-MI-gh2zdo managed
identity doesn't have a subject for the gh-aw-agents environment,
causing AADSTS700213 on every /review command. Revert to repo-level
AZDO_TRIGGER_* secrets until the federated credential is updated.

Fixes the /review trigger broken by #35974.

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

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 36005

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 36005"

kubaflo
kubaflo previously approved these changes Jun 18, 2026
@PureWeen PureWeen closed this Jun 18, 2026
@PureWeen
PureWeen deleted the pureween/fuzzy-goggles branch June 18, 2026 20:15
@PureWeen
PureWeen restored the pureween/fuzzy-goggles branch June 18, 2026 20:15
@PureWeen PureWeen reopened this Jun 18, 2026
@github-actions github-actions Bot added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Jun 18, 2026
@PureWeen
PureWeen force-pushed the pureween/fuzzy-goggles branch 2 times, most recently from 3977da7 to 119d8aa Compare June 18, 2026 20:16
@PureWeen
PureWeen merged commit 32d494c into main Jun 18, 2026
4 of 5 checks passed
@PureWeen
PureWeen deleted the pureween/fuzzy-goggles branch June 18, 2026 20:25
@github-actions github-actions Bot added this to the .NET 10 SR9 milestone Jun 18, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants