Skip to content

Commit b63fc0d

Browse files
enable sm vllm route
1 parent e774df9 commit b63fc0d

File tree

3 files changed

+10
-3
lines changed

3 files changed

+10
-3
lines changed

src/start_testbuilds.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -302,7 +302,8 @@ def main():
302302
# run_test_job(commit, f"dlc-pr-{test_type}", images_str)
303303

304304
# Trigger sagemaker local test jobs when there are changes in sagemaker_tests
305-
if test_type == "sagemaker" and config.is_sm_local_test_enabled():
305+
# Skip SageMaker local tests for vLLM images as they use different test structure
306+
if test_type == "sagemaker" and config.is_sm_local_test_enabled() and "vllm" not in images_str:
306307
test_job = f"dlc-pr-{test_type}-local-test"
307308
run_test_job(commit, test_job, images_str)
308309

test/testrunner.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,13 +327,19 @@ def main():
327327
"security_sanity",
328328
"eks",
329329
"ec2",
330-
"sagemaker",
331330
}:
332331
LOGGER.info(
333332
f"NOTE: {specific_test_type} tests not supported on vllm images. Skipping..."
334333
)
335334
return
336335

336+
# Skip SageMaker tests for vLLM images as they use different test structure
337+
if all("vllm" in image_uri for image_uri in all_image_list) and test_type == "sagemaker":
338+
LOGGER.info(
339+
f"NOTE: SageMaker tests for vLLM images use different test structure. Skipping standard SageMaker tests..."
340+
)
341+
return
342+
337343
# quick_checks tests don't have images in it. Using a placeholder here for jobs like that
338344
try:
339345
framework, version = get_framework_and_version_from_tag(all_image_list[0])

vllm/buildspec-sm.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ images:
4343
latest_release_tag: !join [ *VERSION, "-", *DEVICE_TYPE, "-", *TAG_PYTHON_VERSION, "-", *CUDA_VERSION, "-", *OS_VERSION, "-sm" ]
4444
docker_file: !join [ *FRAMEWORK, /, *ARCH_TYPE, /, *DEVICE_TYPE, /Dockerfile ]
4545
target: vllm-sagemaker
46-
build: true
46+
build: false
4747
enable_common_stage_build: false
4848
test_configs:
4949
test_platforms:

0 commit comments

Comments
 (0)