Skip to content

Conversation

@mmanzoorTT
Copy link
Contributor

Ticket

closes #2192

Problem description

Existing tests assume a single chip, but there’s no clear separation between single-chip and multi-chip tests.

What's changed

  • Created a dedicated directory for single-chip tests.
  • Updated CI JSON files to reference the new paths for single-chip tests.

Checklist

  • Existing tests provide coverage for changes

Copy link
Contributor

@AleksKnezevic AleksKnezevic left a comment

Choose a reason for hiding this comment

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

I would rather not have the tests in separate places. There's no need that a test that we have for N150 shouldn't also work on llmbox in data-parallel. Let's keep all the tests in one place and use @pytest.mark.single_device and @pytest.mark.llmbox

@mmanzoorTT
Copy link
Contributor Author

I would rather not have the tests in separate places. There's no need that a test that we have for N150 shouldn't also work on llmbox in data-parallel. Let's keep all the tests in one place and use @pytest.mark.single_device and @pytest.mark.llmbox

@AleksKnezevic Will data-parallel test for N300 works with @pytest.mark.llmbox?

@AleksKnezevic
Copy link
Contributor

@mmanzoorTT, see pytest.ini there are marks for dual_chip and data_parallel, so we should be able to support it. Here's how I set up the TP tests on llm box: #1745

@github-actions
Copy link
Contributor

github-actions bot commented Nov 14, 2025

TestsPassed ✅Skipped ⚠️Failed
TT-XLA Tests179 ran157 passed22 skipped0 failed
TestResult
No test annotations available

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.

[vLLM plugin] Update testing to support single/multi chip tests

3 participants