[SPARK-50493][BUILD] Upgrade kubernetes-client to 7.0.0 #49066
Conversation
| @@ -37,7 +37,7 @@ | |||
| <dependencies> | |||
| <dependency> | |||
| <groupId>io.fabric8</groupId> | |||
| <artifactId>volcano-model-v1beta1</artifactId> | |||
| <artifactId>volcano-model</artifactId> | |||
There was a problem hiding this comment.
| @@ -227,7 +235,7 @@ netty-transport-native-kqueue/4.1.115.Final/osx-x86_64/netty-transport-native-kq | |||
| netty-transport-native-unix-common/4.1.115.Final//netty-transport-native-unix-common-4.1.115.Final.jar | |||
| netty-transport/4.1.115.Final//netty-transport-4.1.115.Final.jar | |||
| objenesis/3.3//objenesis-3.3.jar | |||
| okhttp/3.12.12//okhttp-3.12.12.jar | |||
| okhttp/4.12.0//okhttp-4.12.0.jar | |||
There was a problem hiding this comment.
The K8s client "Switch default HTTP client from OkHttp to Vert.x" in fabric8io/kubernetes-client#6470
instead of upgrading OkHttp4 and pulling Kotlin runtime deps, maybe following the upstream to switch to Vert.x is a better option?
There was a problem hiding this comment.
Thanks! fine to me.
From 6.x to 7.x, we need to make some changes first, let's run GA first.
There was a problem hiding this comment.
I'm +1 for @pan3793 's comment.
| kotlin-stdlib-common/1.8.21//kotlin-stdlib-common-1.8.21.jar | ||
| kotlin-stdlib-jdk7/1.8.21//kotlin-stdlib-jdk7-1.8.21.jar | ||
| kotlin-stdlib-jdk8/1.8.21//kotlin-stdlib-jdk8-1.8.21.jar | ||
| kotlin-stdlib/1.8.21//kotlin-stdlib-1.8.21.jar |
Well, yes, I'm working on it and it will take some time. |
|
I have already created a new PR #49159 for this feature (switch |
What changes were proposed in this pull request?
The pr aims to upgrade
kubernetes-clientfrom6.13.2to7.0.0.Why are the changes needed?
Fix Move to use okhttp4 due to a bug on IPV6 based cluster fabric8io/kubernetes-client#2632: Bumped OkHttp from 3.12.12 to 4.12.0
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.
Was this patch authored or co-authored using generative AI tooling?
No.