Skip to content

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

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#5044
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 balanced 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
@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

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 support for per-agent-pool overrides to the PR job matrix batch size, allowing different pools to process different numbers of PackageInfo entries per PR job while keeping a default fallback.

Changes:

  • Introduces PRJobBatchSizeByPool parameter for pool-specific override values (including runtime macros).
  • Adds PowerShell logic to resolve the effective batch size per pool and validate it.
  • Updates PR matrix generation to pass the resolved $batchSize to Create-PrJobMatrix.ps1.

💡 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
Comment thread eng/common/pipelines/templates/jobs/generate-job-matrix.yml
Comment thread eng/common/pipelines/templates/jobs/generate-job-matrix.yml

@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:39
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 525ea21 to d59ae02 Compare August 12, 2026 20:39
@azure-sdk-automation
azure-sdk-automation Bot merged commit bc0e453 into main Aug 12, 2026
13 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