Skip to content

Enforce CFSClean network isolation in 1ES pipelines - #852

Open
Mike Harder (mikeharder) with Copilot wants to merge 4 commits into
mainfrom
copilot/port-pr-504-to-repo
Open

Enforce CFSClean network isolation in 1ES pipelines#852
Mike Harder (mikeharder) with Copilot wants to merge 4 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.

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