nit: update comments and use uv cache for pip install if applicable

nv-tusharma · nv-tusharma · commit 1f03876ac69a · 2025-07-14T10:55:55.000-07:00
diff --git a/container/Dockerfile.sglang b/container/Dockerfile.sglang
@@ -431,8 +431,10 @@ RUN apt-get update && \
 
 # Install SGLang and related packages (sgl-kernel, einops, sentencepiece) since they are not included in the runtime wheel
 # https://github.com/sgl-project/sglang/blob/v0.4.9.post1/python/pyproject.toml#L18-51
+# This commit references the 0.4.9.post1 release which includes a NIXL fix that was released after the 0.4.8.post1 release https://github.com/sgl-project/sglang/pull/7330
 ARG SGLANG_COMMIT="066f4ec91f7becd1d490745c856c4a57ede9d1ca"
-RUN uv pip install "sglang[runtime_common]>=0.4.9.post1" einops sgl-kernel==0.2.4 sentencepiece
+RUN --mount=type=cache,target=/root/.cache/uv \
+    uv pip install "sglang[runtime_common]==0.4.9.post1" einops sgl-kernel==0.2.4 sentencepiece
 
 # Install the wheels and symlink executables to /usr/local/bin so dynamo components can use them
 # Dynamo components currently do not have the VIRTUAL_ENV in their PATH, so we need to symlink the executables
@@ -448,7 +450,7 @@ RUN --mount=type=bind,source=./container/launch_message.txt,target=/workspace/la
     sed '/^#\s/d' /workspace/launch_message.txt > ~/.launch_screen && \
     echo "cat ~/.launch_screen" >> ~/.bashrc
 
-# Copy benchmarks, examples, tests and attribution files
+# Copy benchmarks, examples, and tests for CI
 # TODO: Remove this once we have a functional CI image built on top of the runtime image
 COPY tests /workspace/tests
 COPY benchmarks /workspace/benchmarks
