Skip to content

Comments

[Bugfix] Remove Mooncake/Yuanrong connector import warning#1091

Merged
hsliuustc0106 merged 1 commit intovllm-project:mainfrom
natureofnature:bugfix/remove_connector_warning
Jan 30, 2026
Merged

[Bugfix] Remove Mooncake/Yuanrong connector import warning#1091
hsliuustc0106 merged 1 commit intovllm-project:mainfrom
natureofnature:bugfix/remove_connector_warning

Conversation

@natureofnature
Copy link
Contributor

@natureofnature natureofnature commented Jan 30, 2026

PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.

Purpose

Hide warning info for Mooncake/Yuanrong connector if they are not being used. When being used and relevant packages not existing, raise error directly.

Test Plan

N/A

Test Result

Before
image

After
image


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.

BEFORE SUBMITTING, PLEASE READ https://github.com/vllm-project/vllm-omni/blob/main/CONTRIBUTING.md (anything written below this line will be removed by GitHub Actions)

@david6666666
Copy link
Collaborator

please paste test result log

Copy link
Collaborator

@hsliuustc0106 hsliuustc0106 left a comment

Choose a reason for hiding this comment

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

add test log difference

@natureofnature
Copy link
Contributor Author

add test log difference

added

@david6666666 david6666666 added the ready label to trigger buildkite CI label Jan 30, 2026
@david6666666 david6666666 enabled auto-merge (squash) January 30, 2026 04:16
auto-merge was automatically disabled January 30, 2026 04:55

Head branch was pushed to by a user without write access

@natureofnature natureofnature force-pushed the bugfix/remove_connector_warning branch from 7c484d8 to 1058956 Compare January 30, 2026 04:55
@hsliuustc0106 hsliuustc0106 merged commit 790e34e into vllm-project:main Jan 30, 2026
7 checks passed
dongbo910220 pushed a commit to dongbo910220/vllm-omni that referenced this pull request Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready label to trigger buildkite CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants