Skip to content

Enforce CFSClean network isolation in 1ES pipelines - #852

Merged
Mike Harder (mikeharder) merged 5 commits into
mainfrom
copilot/port-pr-504-to-repo
Aug 10, 2026
Merged

Enforce CFSClean network isolation in 1ES pipelines#852
Mike Harder (mikeharder) merged 5 commits into
mainfrom
copilot/port-pr-504-to-repo

Conversation

Copilot AI commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Co-authored-by: mikeharder <9459391+mikeharder@users.noreply.github.com>
@mikeharder Mike Harder (mikeharder) changed the title Porting PR #504 to the new repository Enforce CFSClean network isolation in 1ES pipelines Aug 5, 2026
@mikeharder
Mike Harder (mikeharder) marked this pull request as ready for review August 5, 2026 03:22
@mikeharder
Mike Harder (mikeharder) enabled auto-merge (squash) August 5, 2026 03:22
@mikeharder

Copy link
Copy Markdown
Member

Copilot: Also port this PR from the other repo, into this PR itself:

Azure/openapi-diff#488

Co-authored-by: mikeharder <9459391+mikeharder@users.noreply.github.com>

Copilot AI commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Copilot: Also port this PR from the other repo, into this PR itself:

Azure/openapi-diff#488

Ported PR #488 in commit a248ef2, including the reusable authenticated npmrc template and its use by the shared Rush build setup.

@mikeharder

Copy link
Copy Markdown
Member

Co-authored-by: mikeharder <9459391+mikeharder@users.noreply.github.com>

Copilot AI commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Copilot: fix the error in this pipeline:

https://dev.azure.com/azure-sdk/public/_build/results?buildId=6660178&view=logs&j=3dc8fd7e-4368-5a92-293e-d53cefc8c4b3&t=3c663f50-2031-5e5a-5a2...

Fixed in 7d50674 by routing the regression test’s dynamic AutoRest core resolution through the authenticated npmrc and Microsoft package-feed proxy.

@mikeharder

Copy link
Copy Markdown
Member

Co-authored-by: mikeharder <9459391+mikeharder@users.noreply.github.com>

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Copilot: try porting the technique used in this PR to install pnpm:

Azure/autorest#5194
...

Fixed in e8c1fa4 by installing the pnpm version pinned in rush.json through the authenticated npm configuration before Rush runs.

@mikeharder
Mike Harder (mikeharder) merged commit 6243cb0 into main Aug 10, 2026
11 of 12 checks passed
@mikeharder
Mike Harder (mikeharder) deleted the copilot/port-pr-504-to-repo branch August 10, 2026 21:48
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