Skip to content

[CI] Fix wrong path test file, missing rlhf_async_new_apis.py#37532

Merged
vllm-bot merged 1 commit intovllm-project:mainfrom
EmbeddedLLM:fixdisttest
Mar 19, 2026
Merged

[CI] Fix wrong path test file, missing rlhf_async_new_apis.py#37532
vllm-bot merged 1 commit intovllm-project:mainfrom
EmbeddedLLM:fixdisttest

Conversation

@tjtanaa
Copy link
Copy Markdown
Collaborator

@tjtanaa tjtanaa commented Mar 19, 2026

Purpose

Fix https://buildkite.com/vllm/ci/builds/56931/steps/canvas?sid=019d03bb-7ad8-4f4e-bef3-7599fb7aae29&tab=output
caused by this PR #36188

python3: can't open file '/vllm-workspace/examples/offline_inference/new_weight_syncing/rlhf_async_new_apis.py': [Errno 2] No such file or directory

Test Plan

Test Result

The Test Group passed now
https://buildkite.com/vllm/ci/builds/56972/steps/canvas?sid=019d0516-a1e8-46dc-8380-dc5a92eb7a4e&tab=output


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

Signed-off-by: tjtanaa <tunjian.tan@embeddedllm.com>
@mergify mergify bot added the ci/build label Mar 19, 2026
@tjtanaa tjtanaa added the ready ONLY add when PR is ready to merge/full CI is needed label Mar 19, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request corrects a file path in the Buildkite CI configuration. The path to rlhf_async_new_apis.py was updated to reflect its new location in examples/rl/, which should resolve a failing test step. The change is correct and necessary.

@DarkLight1337 DarkLight1337 enabled auto-merge (squash) March 19, 2026 07:58
@vllm-bot vllm-bot merged commit da70c87 into vllm-project:main Mar 19, 2026
30 of 32 checks passed
fxdawnn pushed a commit to fxdawnn/vllm that referenced this pull request Mar 19, 2026
chooper26 pushed a commit to intellistream/vllm-hust that referenced this pull request Mar 21, 2026
SouthWest7 pushed a commit to SouthWest7/vllm that referenced this pull request Mar 27, 2026
…-project#37532)

Signed-off-by: tjtanaa <tunjian.tan@embeddedllm.com>
khairulkabir1661 pushed a commit to khairulkabir1661/vllm that referenced this pull request Mar 27, 2026
Monishver11 pushed a commit to Monishver11/vllm that referenced this pull request Mar 27, 2026
…-project#37532)

Signed-off-by: tjtanaa <tunjian.tan@embeddedllm.com>
Signed-off-by: Monishver Chandrasekaran <monishverchandrasekaran@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants