Skip to content

Commit 234aa1d

Browse files
remove cleanup logic
1 parent f287b96 commit 234aa1d

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

test/vllm/sagemaker/test_sm_endpoint.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ def test_vllm_on_sagemaker(image_uri, endpoint_name):
165165
print("=" * 80)
166166
# Cleanup any partially created resources
167167
delete_endpoint(endpoint_name)
168-
return
168+
raise Exception("SageMaker endpoint deployment failed")
169169

170170
print("\n" + "-" * 80)
171171
print("PHASE 2: WAITING FOR ENDPOINT READINESS".center(80))
@@ -176,7 +176,7 @@ def test_vllm_on_sagemaker(image_uri, endpoint_name):
176176
print("\n" + "=" * 80)
177177
print("ENDPOINT READINESS FAILED".center(80))
178178
print("=" * 80)
179-
return
179+
raise Exception("SageMaker endpoint failed to become ready")
180180

181181
print("\nEndpoint is ready for inference!")
182182
print("\n" + "-" * 80)
@@ -216,3 +216,4 @@ def test_vllm_on_sagemaker(image_uri, endpoint_name):
216216
print("\n" + "=" * 80)
217217
print(" TEST FAILED ".center(80))
218218
print("=" * 80)
219+
raise Exception("SageMaker endpoint inference test failed")

vllm/x86_64/gpu/Dockerfile

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,11 +39,11 @@ RUN chmod +x /usr/local/bin/deep_learning_container.py && \
3939
# create symlink for python
4040
ln -s /usr/bin/python3 /usr/bin/python && \
4141
# clean up
42-
rm -rf ${HOME_DIR}/oss_compliance* && \
43-
rm -rf /tmp/tmp* && \
44-
rm -rf /tmp/uv* && \
45-
rm -rf /var/lib/apt/lists/* && \
46-
rm -rf /root/.cache | true
42+
# rm -rf ${HOME_DIR}/oss_compliance* && \
43+
# rm -rf /tmp/tmp* && \
44+
# rm -rf /tmp/uv* && \
45+
# rm -rf /var/lib/apt/lists/* && \
46+
# rm -rf /root/.cache | true
4747

4848

4949
RUN bash install_efa.sh ${EFA_VERSION} && \

0 commit comments

Comments
 (0)