Skip to content

Keyless Build Promotion publishing (re-apply #17185) — gated on null-guard deploy#17189

Draft
missymessa wants to merge 2 commits into
dotnet:mainfrom
missymessa:keyless-publish-only
Draft

Keyless Build Promotion publishing (re-apply #17185) — gated on null-guard deploy#17189
missymessa wants to merge 2 commits into
dotnet:mainfrom
missymessa:keyless-publish-only

Conversation

@missymessa

Copy link
Copy Markdown
Member

Keyless Build Promotion publishing (re-apply #17185)

Removes AzureDevOpsFeedsKey and the enable-cross-org-publishing.ps1 -token step from eng/publishing/v3/publish.yml. Publishing runs under the maestro-build-promotion federated identity (Entra/WIF) instead of the dn-bot-all-orgs-artifact-feeds-rw PAT.

This re-applies #17185, which was reverted by #17186 after it broke production Build Promotion.

⚠️ DO NOT MERGE until the null-guard task is deployed to Build Promotion

The publishing task ships in the arcade SDK pinned by global.json. Keyless publishing is only safe once the fixed task (null-guard from the companion PR) is the version Build Promotion actually runs. Required order:

  1. Revert #17185: restore AzureDevOpsFeedsKey (task NREs on null FeedKeys, breaking prod promotions) #17186 (revert) merged — prod unblocked. ✅
  2. Null-guard PR merged (companion PR).
  3. New arcade SDK built with the fix; global.json bumped so Build Promotion consumes it.
  4. Only then merge this PR.

Merging this before the fixed task is live will reproduce the #17185 regression. Kept as draft until the gate above is satisfied.

Context

Copilot AI review requested due to automatic review settings July 24, 2026 22:52

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 wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants