Skip to content

[Test] Skip MOSS TTS offline E2E tests (issue #4700) - #4701

Merged
Gaohan123 merged 1 commit into
vllm-project:mainfrom
yenuo26:skip
Jun 25, 2026
Merged

Gaohan123 merged 1 commit into
vllm-project:mainfrom
yenuo26:skip

Conversation

@yenuo26

@yenuo26 yenuo26 commented Jun 25, 2026 •

Copy link
Copy Markdown
Collaborator

PLEASE FILL IN THE PR DESCRIPTION HERE.

Purpose

MOSS TTS offline E2E tests fail during orchestrator initialization on CI:

  • test_moss_tts.py: OpenMOSS-Team/MOSS-VoiceGenerator requires trust_remote_code=True at ModelConfig validation time.
  • test_moss_tts_realtime.py: Transformers does not recognize architecture moss_tts_realtime.

Skip the affected modules at collection time until the underlying loading/config issues are fixed.

Test Plan

  • No new tests; CI-only skip markers on existing modules.
  • Local collection (with full deps):
pytest tests/e2e/offline_inference/test_moss_tts.py \
       tests/e2e/offline_inference/test_moss_tts_realtime.py --collect-only -q

Expected: 5 tests reported as skipped with reason issues/4700.

vLLM Version: N/A (test-only change)

vLLM-Omni Commit: d6966b12

Test Result

  • Pending CI (Buildkite L2/L3 merge jobs that include MOSS TTS offline inference).
  • Skip markers follow existing offline_inference convention (module-level pytestmark.skip with issue URL).

BEFORE SUBMITTING: read CONTRIBUTING.md and run the precheck-pr skill with the code agent for a self-check against project conventions.
(anything written below this line will be removed by GitHub Actions)

Signed-off-by: wangyu <410167048@qq.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@Gaohan123 Gaohan123 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Gaohan123
Gaohan123 merged commit f6e8b77 into vllm-project:main Jun 25, 2026
4 of 5 checks passed
khairulkabir1661 pushed a commit to khairulkabir1661/vllm-omni that referenced this pull request Sep 25, 2026
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.

2 participants