From b3252801c1c99553fccf90614519278e1b6957c5 Mon Sep 17 00:00:00 2001 From: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com> Date: Mon, 6 Jul 2026 14:30:27 -0700 Subject: [PATCH] [nvbugs/6418006][fix] Replace Triton test_whisper with cleanup assertion PR #15763 removed examples/models/core/whisper/ as part of the legacy TensorRT-backend cleanup, but tests/integration/defs/triton_server/build_model.sh still ran 'pushd examples/models/core/whisper' and invoked convert_checkpoint.py for the whisper target, causing the Triton post-merge test to fail with 'pushd: examples/models/core/whisper: No such file or directory'. Replace the body of test_whisper with a lightweight assertion that the removed convert_checkpoint.py script has not been reintroduced. This keeps the pytest node id resolvable (the harness runs the test by exact node id) while removing the broken build/test.sh invocation. Mirrors the pattern used for test_medusa in nvbugs/6418017. The dead 'whisper' branches in build_model.sh and test.sh are no longer reachable and can be pruned in a follow-up cleanup. Signed-off-by: trtllm-agent <296075020+trtllm-agent@users.noreply.github.com> --- .../defs/triton_server/test_triton.py | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/tests/integration/defs/triton_server/test_triton.py b/tests/integration/defs/triton_server/test_triton.py index 6d710ad5ba1d..f51fda984fe7 100644 --- a/tests/integration/defs/triton_server/test_triton.py +++ b/tests/integration/defs/triton_server/test_triton.py @@ -312,15 +312,16 @@ def test_eagle(tritonserver_test_root, test_name, llm_root, model_path, @pytest.mark.parametrize("test_name", ["whisper"], indirect=True) -def test_whisper(tritonserver_test_root, test_name, llm_root, model_path, - engine_dir): - build_model(test_name, llm_root, tritonserver_test_root) - tokenizer_type = "auto" - decoder_path = f"{engine_dir}/decoder" - encoder_path = f"{engine_dir}/encoder" - run_shell_command( - f"cd {tritonserver_test_root} && ./test.sh {test_name} {decoder_path} {model_path} {tokenizer_type} skip skip {encoder_path}", - llm_root) +def test_whisper(tritonserver_test_root, test_name, llm_root): + # examples/models/core/whisper/convert_checkpoint.py was removed with the + # legacy TensorRT-backend cleanup; guard against silent reintroduction. + whisper_convert_script = os.path.join(llm_root, "examples", "models", + "core", "whisper", + "convert_checkpoint.py") + assert not os.path.exists(whisper_convert_script), ( + f"{whisper_convert_script} was reintroduced; update this test or " + "restore the Triton whisper build pipeline in build_model.sh and " + "test.sh.") @pytest.mark.parametrize("test_name", ["t5-ib"], indirect=True)