Skip to content

[CI] Shard Humming H100 eval - #52326

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

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

Conversation

@khluu

@khluu khluu commented Aug 14, 2026 •

Copy link
Copy Markdown
Member

Why

lm-eval-humming-f16-h100 took 62.6 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/H100 sharding terms before implementation.

What changed

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

The measured baseline loads are 20.52, 20.38, and 20.74 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 H100 jobs with the image-build dependency
  • targeted build 83895: all three shards passed, with exact 14/14 configuration coverage and no omissions or duplicates
  • targeted shard wall times: 20.79, 20.33, and 19.33 minutes (1.48-minute max/min spread; 1.08x imbalance)
  • outer job/bootstrap + teardown overhead around pytest: 0.83, 0.80, and 0.74 minutes per shard
  • critical path: 20.79 minutes versus 62.63 minutes in build 83851 (3.01x faster)
  • observed total GPU time: 60.45 versus 62.63 GPU-minutes in the single baseline run; the -2.18 minute delta is normal run-to-run variance, not negative sharding overhead
  • 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-h100 branch from 9c2377b to 6495a6e Compare August 14, 2026 11:37
@khluu
khluu marked this pull request as ready for review August 14, 2026 19:25

@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 bb4b448 into vllm-project:main Aug 14, 2026
13 checks passed
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>
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