Skip to content

Commit

Permalink
[Docker] Remove Kubectl (ray-project#38524)
Browse files Browse the repository at this point in the history
Ray no longer needs kubectl. It was used for the old "kubernetes cluster launcher." KubeRay does not require that Kubectl is installed to run.

Signed-off-by: Ian Rodney <[email protected]>
  • Loading branch information
ijrsvt authored Sep 11, 2023
1 parent e69e888 commit b57390a
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions docker/base-deps/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -78,15 +78,6 @@ RUN sudo apt-get update -y && sudo apt-get upgrade -y \
# We keep g++ on GPU images, because uninstalling removes CUDA Devel tooling
$(if [[ "$BASE_IMAGE" == "ubuntu:focal" && "$HOSTTYPE" == "x86_64" ]]; then echo \
g++; fi) \
# Either install kubectl or remove wget
&& (if [ "$AUTOSCALER" = "autoscaler" ]; \
then wget -O - -q https://packages.cloud.google.com/apt/doc/apt-key.gpg | sudo apt-key add - \
&& sudo touch /etc/apt/sources.list.d/kubernetes.list \
&& echo "deb http://apt.kubernetes.io/ kubernetes-xenial main" | sudo tee -a /etc/apt/sources.list.d/kubernetes.list \
&& sudo apt-get update \
&& sudo apt-get install kubectl; \
else sudo apt-get autoremove -y wget; \
fi;) \
&& sudo rm -rf /var/lib/apt/lists/* \
&& sudo apt-get clean

Expand Down

0 comments on commit b57390a

Please sign in to comment.