Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16674 - #61929

Merged
azure-sdk-automation[bot] merged 2 commits into
mainfrom
sync-eng-common-weikanglim/pr-job-batch-size-by-pool-16674
Aug 12, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 16674#61929
azure-sdk-automation[bot] merged 2 commits into
mainfrom
sync-eng-common-weikanglim/pr-job-batch-size-by-pool-16674

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner August 11, 2026 18:06
@azure-sdk-automation
azure-sdk-automation Bot requested review from Wei Lim (weikanglim) and a lite review from Copilot August 11, 2026 18:06
@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 11, 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 pool-specific overrides for the PR job matrix batch size so different agent pools can generate PR job matrices with different “packages per job” values.

Changes:

  • Introduces PRJobBatchSizeByPool parameter to override PRJobBatchSize per pool.
  • Adds PowerShell logic to resolve per-pool overrides (including runtime-variable handling) and validate the final batch size.
  • Passes the resolved $batchSize into Create-PrJobMatrix.ps1 instead of the global default.

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

Comment thread eng/common/pipelines/templates/jobs/generate-job-matrix.yml
Comment thread eng/common/pipelines/templates/jobs/generate-job-matrix.yml
@azure-pipelines

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

@weikanglim Wei Lim (weikanglim) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The synchronized eng/common change is ready. Required checks are passing and all review threads have been addressed.

Wei Lim (weikanglim) and others added 2 commits August 12, 2026 20:37
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@azure-sdk-automation
azure-sdk-automation Bot force-pushed the sync-eng-common-weikanglim/pr-job-batch-size-by-pool-16674 branch from e2fb9dd to 17c9865 Compare August 12, 2026 20:37
@azure-sdk-automation
azure-sdk-automation Bot merged commit 23591ab into main Aug 12, 2026
24 checks passed
@azure-sdk-automation
azure-sdk-automation Bot deleted the sync-eng-common-weikanglim/pr-job-batch-size-by-pool-16674 branch August 12, 2026 22:35
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.

3 participants