Support to get spark app url with pattern http://{{SPARK_DRIVER_POD_IP}}:{{SPARK_UI_PORT}}#7141
Support to get spark app url with pattern http://{{SPARK_DRIVER_POD_IP}}:{{SPARK_UI_PORT}}#7141turboFei wants to merge 3 commits intoapache:masterfrom
http://{{SPARK_DRIVER_POD_IP}}:{{SPARK_UI_PORT}}#7141Conversation
http://{{SPARK_DRIVER_POD_IP}}:{{SPARK_UI_PORT}}
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #7141 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 700 701 +1
Lines 43454 43500 +46
Branches 5883 5895 +12
======================================
- Misses 43454 43500 +46 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
kyuubi-server/src/main/scala/org/apache/kyuubi/engine/KubernetesApplicationOperation.scala
Outdated
Show resolved
Hide resolved
…esApplicationOperation.scala
|
LGTM, one question, SPARK-42769 is only available since Spark 3.5, do we need to write something in docs for these requirements? |
Not necessary, this PR can also work with any spark version that supports spark on kubernetes. SPARK-42769 is only for executors to connect to driver via IP. |
|
Have updated the PR desc. |
pan3793
left a comment
There was a problem hiding this comment.
I see, Pod IP is retrieved by K8s client
Why are the changes needed?
We are using virtual-kubelet for spark on kubernetes, and spark kubernetes pods would be allocated across kubernetes clusters.
And we use the driver POD ip as driver host, see apache/spark#40392, which is supported since spark-3.5.
The kubernetes context and namespace are virtual and we can not build the app URL by spark driver svc.
And the spark driver pod IP is accessible for our use case, so raise this PR to build the spark app url by spark driver pod id and spark ui port.
How was this patch tested?
UT.
Was this patch authored or co-authored using generative AI tooling?
No.