Skip to content

[DO NOT MERGE] CI measurement: 100pkg - Set A - #59406

Closed
m-nash wants to merge 4 commits into
mainfrom
test/loc-split-treatment-100pkg
Closed

[DO NOT MERGE] CI measurement: 100pkg - Set A#59406
m-nash wants to merge 4 commits into
mainfrom
test/loc-split-treatment-100pkg

Conversation

@m-nash

@m-nash m-nash commented May 21, 2026

Copy link
Copy Markdown
Member

Set A (treatment vs feature branch) for measuring the standalone build/analyze LOC-split change.

  • Scale: 100pkg (99 packages touched - 1 file dropped due to upstream deletion)
  • Base: feature/split-build-analyze-loc
  • Purpose: measure CI build/analyze wall-clock and agent-min for a clean A/B against the LOC-split feature only (no other weighted-test-batching changes).

Companion to PR set in the other direction. Do not merge.

m-nash and others added 2 commits May 21, 2026 15:16
Replaces single batched-build-analyze matrix with two parallel matrix-gens, each tuned independently:

- Build: LocTargetSeconds=1,200,000 (larger batches, fewer jobs, lower fixed-overhead)

- Analyze: LocTargetSeconds=300,000 (smaller batches, more parallelism)

Adds pr-matrix-presteps.yml params LocTargetSeconds (default 474000) and PublishArtifact (default true) so each matrix-gen can configure LOC target independently and only the first publishes BuildPackagesArtifact (avoiding duplicate-artifact collision).

Adds Get-PackageLocWeights.ps1 (LOC counter) and Apply-WeightedBatching.ps1 (LPT bin-packing) which together drive the LOC-weighted bin-packing. Removes the now-unused batched-build-analyze.yml.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

Hi m-nash. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Hi m-nash. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing /reopen if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.

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

Projects

Status: Done
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant