Revert "Fix issue where unescaped semicolons caused task execution failures. (#3691)" (#3771) (#3772)

MortalHappiness · web-flow · commit f96de04fb178 · 2025-06-13T14:00:50.000+08:00
This reverts commit 5769a65. Signed-off-by: Chi-Sheng Liu <chishengliu@chishengliu.com>
diff --git a/ray-operator/controllers/ray/common/job.go b/ray-operator/controllers/ray/common/job.go
@@ -116,7 +116,6 @@ func GetK8sJobCommand(rayJobInstance *rayv1.RayJob) ([]string, error) {
 	}
 
 	// "--" is used to separate the entrypoint from the Ray Job CLI command and its arguments.
-	entrypoint = strings.ReplaceAll(entrypoint, ";", "\\;")
 	k8sJobCommand = append(k8sJobCommand, "--", entrypoint, ";", "fi", ";")
 	k8sJobCommand = append(k8sJobCommand, jobFollowCommand...)
 

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,6 @@ func GetK8sJobCommand(rayJobInstance *rayv1.RayJob) ([]string, error) {`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`// "--" is used to separate the entrypoint from the Ray Job CLI command and its arguments.`
`119`		`- entrypoint = strings.ReplaceAll(entrypoint, ";", "\\;")`
`120`	`119`	`k8sJobCommand = append(k8sJobCommand, "--", entrypoint, ";", "fi", ";")`
`121`	`120`	`k8sJobCommand = append(k8sJobCommand, jobFollowCommand...)`
`122`	`121`