kuberay autoscaler pod use same command and args as ray head container (

ray-project#2268) * kuberay autoscaler pod use same command and args like main container Signed-off-by: zheng <[email protected]> * add new container args to pod test --------- Signed-off-by: zheng <[email protected]> Co-authored-by: 王正 <[email protected]>
kevin85421 · Aug 4, 2024 · a6816ff · a6816ff
1 parent 6c168a0
commit a6816ff
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 12 deletions.
diff --git a/ray-operator/controllers/ray/common/pod.go b/ray-operator/controllers/ray/common/pod.go
@@ -429,14 +429,12 @@ func BuildAutoscalerContainer(autoscalerImage string) corev1.Container {
 			},
 		},
 		Command: []string{
-			"ray",
+			"/bin/bash",
+			"-lc",
+			"--",
 		},
 		Args: []string{
-			"kuberay-autoscaler",
-			"--cluster-name",
-			"$(RAY_CLUSTER_NAME)",
-			"--cluster-namespace",
-			"$(RAY_CLUSTER_NAMESPACE)",
+			"ray kuberay-autoscaler --cluster-name $(RAY_CLUSTER_NAME) --cluster-namespace $(RAY_CLUSTER_NAMESPACE)",
 		},
 		Resources: corev1.ResourceRequirements{
 			Limits: corev1.ResourceList{

diff --git a/ray-operator/controllers/ray/common/pod_test.go b/ray-operator/controllers/ray/common/pod_test.go
@@ -192,14 +192,12 @@ var autoscalerContainer = corev1.Container{
 		},
 	},
 	Command: []string{
-		"ray",
+		"/bin/bash",
+		"-lc",
+		"--",
 	},
 	Args: []string{
-		"kuberay-autoscaler",
-		"--cluster-name",
-		"$(RAY_CLUSTER_NAME)",
-		"--cluster-namespace",
-		"$(RAY_CLUSTER_NAMESPACE)",
+		"ray kuberay-autoscaler --cluster-name $(RAY_CLUSTER_NAME) --cluster-namespace $(RAY_CLUSTER_NAMESPACE)",
 	},
 	Resources: corev1.ResourceRequirements{
 		Limits: corev1.ResourceList{