[Ray 2.3.0] Update --redis-password for RayCluster (ray-project#929)

Ray 2.3.0 changes REDIS_DEFAULT_PASSWORD from "5241590000000000" to ""
lowang-bh · Feb 25, 2023 · 36b72c4 · 36b72c4
1 parent 8d99b06
commit 36b72c4
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 19 deletions.
diff --git a/.github/workflows/test-job.yaml b/.github/workflows/test-job.yaml
@@ -360,13 +360,13 @@ jobs:
         with:
           ray_version: nightly
 
-  sample-yaml-config-test-2_1_0:
+  sample-yaml-config-test-2_3_0:
     needs:
       - build_operator
       - build_apiserver
       - lint
     runs-on: ubuntu-latest
-    name: Sample YAML Config Test - 2.1.0
+    name: Sample YAML Config Test - 2.3.0
     steps:
       - name: Check out code into the Go module directory
         uses: actions/checkout@v2
@@ -377,5 +377,5 @@ jobs:
           ref: ${{github.event.pull_request.head.sha}}
       - uses: ./.github/workflows/actions/configuration
         with:
-          ray_version: 2.1.0
+          ray_version: 2.3.0
 
diff --git a/ray-operator/config/samples/ray-cluster.external-redis.yaml b/ray-operator/config/samples/ray-cluster.external-redis.yaml
@@ -77,20 +77,23 @@ metadata:
   # An unique identifier for the head node and workers of this cluster.
   name: raycluster-external-redis
 spec:
-  rayVersion: '2.2.0'
+  rayVersion: '2.3.0'
   headGroupSpec:
     serviceType: ClusterIP # optional
     replicas: 1
     rayStartParams:
-      dashboard-host: '0.0.0.0'
-      num-cpus: '1' # can be auto-completed from the limits
-      block: 'true'
+      dashboard-host: "0.0.0.0"
+      num-cpus: "1" # can be auto-completed from the limits
+      block: "true"
+      # redis-password should match "requirepass" in redis.conf in the ConfigMap above.
+      # Ray 2.3.0 changes the default redis password from "5241590000000000" to "".
+      redis-password: "5241590000000000"
     #pod template
     template:
       spec:
         containers:
           - name: ray-head
-            image: rayproject/ray:2.2.0
+            image: rayproject/ray:2.3.0
             env:
               # RAY_REDIS_ADDRESS can force ray to use external redis
               - name: RAY_REDIS_ADDRESS
@@ -128,7 +131,7 @@ spec:
               command: ['sh', '-c', "until nslookup $RAY_IP.$(cat /var/run/secrets/kubernetes.io/serviceaccount/namespace).svc.cluster.local; do echo waiting for K8s Service $RAY_IP; sleep 2; done"]
           containers:
             - name: ray-worker
-              image: rayproject/ray:2.2.0
+              image: rayproject/ray:2.3.0
               volumeMounts:
                 - mountPath: /tmp/ray
                   name: ray-logs

diff --git a/ray-operator/config/security/ray-cluster.pod-security.yaml b/ray-operator/config/security/ray-cluster.pod-security.yaml
@@ -26,7 +26,7 @@ spec:
       spec:
         containers:
         - name: ray-head
-          image: rayproject/ray-ml:2.2.0
+          image: rayproject/ray-ml:2.3.0
           ports:
           - containerPort: 6379
             name: gcs
@@ -91,7 +91,7 @@ spec:
       spec:
         containers:
         - name: ray-worker
-          image: rayproject/ray-ml:2.2.0
+          image: rayproject/ray-ml:2.3.0
           # environment variables to set in the container.Optional.
           # Refer to https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/
           lifecycle:

diff --git a/tests/compatibility-test.py b/tests/compatibility-test.py
@@ -31,10 +31,10 @@
 )
 
 # Default Ray version
-ray_version = '2.2.0'
+ray_version = '2.3.0'
 
 # Default docker images
-ray_image = 'rayproject/ray:2.2.0'
+ray_image = 'rayproject/ray:2.3.0'
 kuberay_operator_image = 'kuberay/operator:nightly'
 
 

diff --git a/tests/config/ray-cluster.ray-ft.yaml.template b/tests/config/ray-cluster.ray-ft.yaml.template
@@ -74,9 +74,10 @@ spec:
   rayVersion: '$ray_version'
   headGroupSpec:
     rayStartParams:
-      dashboard-host: '0.0.0.0'
-      num-cpus: '1'
-      block: 'true'
+      dashboard-host: "0.0.0.0"
+      num-cpus: "1"
+      block: "true"
+      redis-password: "5241590000000000"
     #pod template
     template:
       metadata:

diff --git a/tests/test_sample_raycluster_yamls.py b/tests/test_sample_raycluster_yamls.py
@@ -68,7 +68,7 @@
 
     rs = RuleSet([HeadPodNameRule(), EasyJobRule(), HeadSvcRule()])
     image_dict = {
-        CONST.RAY_IMAGE_KEY: os.getenv('RAY_IMAGE', default='rayproject/ray:2.2.0'),
+        CONST.RAY_IMAGE_KEY: os.getenv('RAY_IMAGE', default='rayproject/ray:2.3.0'),
         CONST.OPERATOR_IMAGE_KEY: os.getenv('OPERATOR_IMAGE', default='kuberay/operator:nightly'),
     }
     logger.info(image_dict)

diff --git a/tests/test_sample_rayservice_yamls.py b/tests/test_sample_rayservice_yamls.py
@@ -34,7 +34,7 @@
 
     rs = RuleSet([EasyJobRule(), CurlServiceRule()])
     image_dict = {
-        CONST.RAY_IMAGE_KEY: os.getenv('RAY_IMAGE', default='rayproject/ray:2.2.0'),
+        CONST.RAY_IMAGE_KEY: os.getenv('RAY_IMAGE', default='rayproject/ray:2.3.0'),
         CONST.OPERATOR_IMAGE_KEY: os.getenv('OPERATOR_IMAGE', default='kuberay/operator:nightly'),
     }
     logger.info(image_dict)

diff --git a/tests/test_security.py b/tests/test_security.py
@@ -55,7 +55,7 @@ def setUpClass(cls):
                              {PodSecurityTestCase.namespace}.kubernetes.io/enforce-version=latest")
         # Install the KubeRay operator in the namespace pod-security.
         image_dict = {
-            CONST.RAY_IMAGE_KEY: 'rayproject/ray-ml:2.2.0',
+            CONST.RAY_IMAGE_KEY: 'rayproject/ray-ml:2.3.0',
             CONST.OPERATOR_IMAGE_KEY: os.getenv('OPERATOR_IMAGE','kuberay/operator:nightly'),
         }
         logger.info(image_dict)