Skip to content

revert(pipeline): roll back CFS registry config in setup.yml to unblock releases - #9476

Merged
Elaina-Lee merged 2 commits into
mainfrom
elaina/rollback-cfs-setup
Jul 29, 2026
Merged

revert(pipeline): roll back CFS registry config in setup.yml to unblock releases#9476
Elaina-Lee merged 2 commits into
mainfrom
elaina/rollback-cfs-setup

Conversation

@Elaina-Lee

@Elaina-Lee Elaina-Lee commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Commit Type

  • feature - New functionality
  • fix - Bug fix
  • refactor - Code restructuring without behavior change
  • perf - Performance improvement
  • docs - Documentation update
  • test - Test-related changes
  • chore - Maintenance/tooling

Risk Level

  • Low - Minor changes, limited scope
  • Medium - Moderate changes, some user impact
  • High - Major changes, significant user/system impact

What & Why

The CFS registry routing added in #9453 breaks the ADO release pipeline at the very first install step. "Install pnpm" (npm install -g pnpm) now resolves against the CFS feed and fails with:

npm error code E401
npm error Unable to authenticate, your authentication token seems to be invalid.

so no pipeline run can get past setup. This PR rolls .azure-pipelines/templates/setup.yml back to its exact pre-#9453 state (removes the CFS registry append step, the NpmAuthenticate@0 task, and the npm_config_registry override on the pnpm install) to unblock releases. The SFI Network Isolation (SR21) work can be re-landed once feed authentication is sorted out.

The other two files from #9453 are intentionally left as-is: the 1esmain.yml branch pin was already reverted in #9474, and the .npmrc change was only a trailing-newline fix.

Impact of Change

Test Plan

Contributors

Screenshots/Videos

The CFS registry routing added in #9453 breaks the release pipeline at
the "Install pnpm" step: `npm install -g pnpm` resolves against the CFS
feed and fails with E401 (invalid/missing auth token), so no pipeline
run can get past setup. Roll back setup.yml to its pre-#9453 state to
unblock releases; the SFI/SR21 CFS work can be re-landed once the feed
authentication is sorted out.

The other two files from #9453 stay as they are: the 1esmain.yml branch
pin was already reverted in #9474, and the .npmrc change was only a
trailing-newline fix.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01391HTVsUAx7BaUJUvRgVdV
Copilot AI review requested due to automatic review settings July 29, 2026 22:56
@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

🤖 AI PR Validation Report

PR Review Results

Thank you for your submission! Here's detailed feedback on your PR title and body compliance:

PR Title

  • Current: revert(pipeline): roll back CFS registry config in setup.yml to unblock releases
  • Issue: None — this is a revert PR.
  • Recommendation: No action needed. Revert PRs are auto-approved by policy.

Commit Type

  • This is a revert PR, so template checks are waived.
  • No further action required.

Risk Level

  • Revert PR — risk checks waived. (For reference, the change is pipeline-only and low risk, consistent with the declared Low.)

What & Why


Impact of Change

  • Users, Developers, and System impact are all documented.
  • Recommendation:
    • Users: As stated — none.
    • Developers: As stated — none.
    • System: As stated — ADO pipeline restores from public npm registry again.

Test Plan


Contributors

  • Blank, but not required. Consider crediting reviewers if applicable.

Screenshots/Videos

  • Not applicable — pipeline YAML change with no visual impact.

Summary Table

Section Status Recommendation
Title Revert PR — auto-approved
Commit Type Waived for revert
Risk Level Waived for revert
What & Why Clear and complete
Impact of Change Fully documented
Test Plan Waived for revert
Contributors Optional
Screenshots/Videos Not applicable

This is a revert PR and passes automatically per policy. Nice, well-documented rollback to unblock releases! ✅


Powered by: Copilot CLI (claude-opus-4.8) | Last updated: Wed, 29 Jul 2026 23:33:37 GMT

@Elaina-Lee
Elaina-Lee enabled auto-merge (squash) July 29, 2026 22:57
@github-actions

Copy link
Copy Markdown
Contributor

📊 Coverage Check

No source files changed in this PR.

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.

Pull request overview

Restores the Azure DevOps release setup to the previously working public npm registry configuration.

Changes:

  • Removes CFS registry configuration and authentication.
  • Removes the CFS override from pnpm installation.

@Elaina-Lee
Elaina-Lee merged commit f5ebff5 into main Jul 29, 2026
16 checks passed
@Elaina-Lee
Elaina-Lee deleted the elaina/rollback-cfs-setup branch July 29, 2026 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants