Skip to content

Commit 9d90c8b

Browse files
committed
[SPARK-30738][K8S] Use specific image version in "Launcher client dependencies" test
### What changes were proposed in this pull request? This PR use a specific version of docker image instead of `latest`. As of today, when I run K8s integration test locally, this test case fails always. Also, in this PR, I shows two consecutive failures with a dummy change. - #27465 (comment) - #27465 (comment) ``` - Launcher client dependencies *** FAILED *** ``` After that, I added the patch and K8s Integration test passed. - #27465 (comment) ### Why are the changes needed? [SPARK-28465](#25222) switched from `v4.0.0-stable-4.0-master-centos-7-x86_64` to `latest` to catch up the API change. However, the API change seems to occur again. We had better use a specific version to prevent accidental failures. ```scala - .withImage("ceph/daemon:v4.0.0-stable-4.0-master-centos-7-x86_64") + .withImage("ceph/daemon:latest") ``` ### Does this PR introduce any user-facing change? No. ### How was this patch tested? Pass `Launcher client dependencies` test in Jenkins K8s Integration Suite. Or, run K8s Integration test locally. Closes #27465 from dongjoon-hyun/SPARK-K8S-IT. Authored-by: Dongjoon Hyun <[email protected]> Signed-off-by: Dongjoon Hyun <[email protected]>
1 parent ec70e07 commit 9d90c8b

File tree

1 file changed

+1
-1
lines changed
  • resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest

1 file changed

+1
-1
lines changed

resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/DepsTestsSuite.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ private[spark] trait DepsTestsSuite { k8sSuite: KubernetesSuite =>
5959
).asJava
6060

6161
new ContainerBuilder()
62-
.withImage("ceph/daemon:latest")
62+
.withImage("ceph/daemon:v4.0.3-stable-4.0-nautilus-centos-7-x86_64")
6363
.withImagePullPolicy("Always")
6464
.withName(cName)
6565
.withPorts(new ContainerPortBuilder()

0 commit comments

Comments
 (0)