Skip to content

[TEST - DO NOT MERGE] CI gating probe: tests/gpu_vllm/conftest.py - #2099

Closed
kevalmorabia97 wants to merge 1 commit into
mainfrom
test-ci-gate-4-gpuvllm
Closed

[TEST - DO NOT MERGE] CI gating probe: tests/gpu_vllm/conftest.py#2099
kevalmorabia97 wants to merge 1 commit into
mainfrom
test-ci-gate-4-gpuvllm

Conversation

@kevalmorabia97

Copy link
Copy Markdown
Collaborator

Temporary draft PR verifying the lane gating from #2090 against a real PR. Do not review or merge — this will be closed and its branch deleted once the gate decisions are recorded.

Change: tests/gpu_vllm/conftest.py

Expected: all 5 gpu suites (gpu gating is per-workflow, not per-suite); no example lanes, no regression

Signed-off-by: Keval Morabia <28916987+kevalmorabia97@users.noreply.github.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 7, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 68d60d2f-9fe3-4b48-92f4-9aeea7ad9f50

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@kevalmorabia97

Copy link
Copy Markdown
Collaborator Author

/ok to test c8da569

@codecov

codecov Bot commented Aug 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.60%. Comparing base (3d4d924) to head (c8da569).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2099   +/-   ##
=======================================
  Coverage   78.60%   78.60%           
=======================================
  Files         522      522           
  Lines       60167    60167           
=======================================
  Hits        47294    47294           
  Misses      12873    12873           
Flag Coverage Δ
unit 55.39% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kevalmorabia97

Copy link
Copy Markdown
Collaborator Author

Verified: all four example lanes skipped, gpu suites ran — matches expectation for a tests/gpu_vllm change.

Closing this probe and deleting its branch.

@kevalmorabia97
kevalmorabia97 deleted the test-ci-gate-4-gpuvllm branch August 7, 2026 07:58
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-08-07 07:58 UTC

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant