Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16727 - #7331

Open
azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-AllowAzModulesAndDogfoodSkip-16727
Open

Sync eng/common directory with azure-sdk-tools for PR 16727#7331
azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-AllowAzModulesAndDogfoodSkip-16727

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#16727 See eng/common workflow

Copilot AI lite review requested due to automatic review settings August 14, 2026 20:39
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner August 14, 2026 20:39
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Aug 14, 2026

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

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Adds a parameterized switch to optionally skip environment setup during test resource deployment to support scenarios where setup is unnecessary or handled elsewhere.

Changes:

  • Introduced a new SkipEnvironmentSetup parameter with a default of false.
  • Wrapped the setup-environments.yml template invocation in a conditional block based on SkipEnvironmentSetup.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

UseFederatedAuth: true
PersistOidcToken: false
SelfContainedPostScript: self-contained-test-resources-post.ps1
SkipEnvironmentSetup: false
Comment on lines +46 to +47
- ${{ if eq(parameters.SkipEnvironmentSetup, false) }}:
- template: /eng/common/TestResources/setup-environments.yml
@azure-pipelines

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants