Skip to content

Modify network isolation policy in CI configuration to Permissive in PRs#55445

Open
marcpopMSFT wants to merge 1 commit into
release/8.0.1xxfrom
marcpopMSFT-patch-2
Open

Modify network isolation policy in CI configuration to Permissive in PRs#55445
marcpopMSFT wants to merge 1 commit into
release/8.0.1xxfrom
marcpopMSFT-patch-2

Conversation

@marcpopMSFT

Copy link
Copy Markdown
Member

Updated network isolation settings for internal PR builds.

Updated network isolation settings for internal PR builds.
Copilot AI review requested due to automatic review settings July 23, 2026 23:28
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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

This PR updates the Azure DevOps CI configuration for internal PR builds to use a permissive network isolation policy under the 1ES Unofficial pipeline template, aligning internal PR behavior with template expectations while accommodating restores from public feeds.

Changes:

  • Changed internal PR build network isolation configuration from networkIsolationMode: None to networkIsolationPolicy: Permissive.
  • Updated the accompanying comment describing why PR builds use a different network isolation setting.

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

Comment thread .vsts-ci.yml
Comment on lines 66 to +70
# Internal PR builds run under the 1ES Unofficial template. They restore from
# public feeds (nuget.org, npm, yarn), so network isolation is not applicable
# and is disabled to avoid spurious CFS clean violations.
# and so we set it to permissive and requested an exception
settings:
networkIsolationMode: None
networkIsolationPolicy: Permissive
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.

2 participants