Skip to content

[CI] Shard Humming A100 eval - #52325

Merged
vllm-bot merged 1 commit into
vllm-project:mainfrom
khluu:ci-shard-humming-a100
Aug 17, 2026
Merged

vllm-bot merged 1 commit into
vllm-project:mainfrom
khluu:ci-shard-humming-a100

Conversation

@khluu

@khluu khluu commented Aug 14, 2026

Copy link
Copy Markdown
Member

Why

lm-eval-humming-f16-a100 took 64.3 minutes in full CI build 83851. Its 14 model configurations ran serially even though each evaluation is independent.

This does not duplicate an open pull request. Open PRs were searched by the exact step key and Humming/A100 sharding terms before implementation.

What changed

  • run the step with three Buildkite jobs
  • partition all 14 configurations into three explicit A100 lists using observed build 83851 per-config durations
  • keep each configuration in exactly one shard

The measured baseline loads are 20.20, 21.47, and 21.60 minutes of test work, leaving headroom below the 30-minute target.

Validation

  • baseline model evaluation: 14/14 configurations passed in build 83851
  • config partition check: 14/14 unique entries, no omissions or duplicates, all referenced YAML files exist
  • selected-step pipeline generation: three parallel A100 jobs with the image-build dependency
  • targeted build 83894: all three shards passed; the executed tests exactly match each shard list and cover all 14/14 configurations with no omissions or duplicates
  • targeted shard wall times: 24.09, 22.07, and 24.44 minutes (2.37-minute max/min spread; 1.11x imbalance)
  • outer job/bootstrap + teardown overhead around pytest: 0.82, 0.81, and 0.82 minutes per shard
  • critical path: 24.44 minutes versus 64.27 minutes in build 83851 (2.63x faster)
  • observed total GPU time: 70.59 versus 64.27 GPU-minutes in the single baseline run (+9.8%)
  • uvx pre-commit run --files ...: passed
  • git diff --check: passed

AI assistance was used. The submitter will review and validate the change before marking it ready.

@mergify mergify Bot added the ci/build label Aug 14, 2026
Assisted-by: AI
Signed-off-by: Kevin Luu <51931015+khluu@users.noreply.github.com>
@khluu
khluu force-pushed the ci-shard-humming-a100 branch from 3761824 to eb888d3 Compare August 14, 2026 11:37
@khluu
khluu marked this pull request as ready for review August 17, 2026 09:02

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@vllm-bot
vllm-bot merged commit bb23362 into vllm-project:main Aug 17, 2026
13 checks passed
zyp2014 pushed a commit to zyp2014/vllm that referenced this pull request Aug 21, 2026
Signed-off-by: Kevin Luu <51931015+khluu@users.noreply.github.com>
wyettzeng pushed a commit to wyettzeng/vllm that referenced this pull request Aug 21, 2026
Signed-off-by: Kevin Luu <51931015+khluu@users.noreply.github.com>
Signed-off-by: Wyett <wyettzeng@gmail.com>
zufangzhu pushed a commit to zufangzhu/vllm that referenced this pull request Aug 24, 2026
Signed-off-by: Kevin Luu <51931015+khluu@users.noreply.github.com>
Signed-off-by: Zhu, Zufang <zufang.zhu@intel.com>
khushali9 pushed a commit to khushali9/vllm that referenced this pull request Aug 29, 2026
Signed-off-by: Kevin Luu <51931015+khluu@users.noreply.github.com>
Signed-off-by: khushali9 <khushali.desai9@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants