GitHub user adelbertc opened a pull request:
https://github.com/apache/spark/pull/21884
k8s: explicitly expose ports on driver container
## What changes were proposed in this pull request?
Explicitly specify exposed container port
Expose ports explicitly in the driver container. The driver Service created
expects to reach the driver Pod at specific ports which before this change,
were not explicitly exposed and would likely cause connection issues (see
https://github.com/apache-spark-on-k8s/spark/issues/617).
This is a port of the original PR created in the now-deprecated Kubernetes
fork: https://github.com/apache-spark-on-k8s/spark/pull/618
## How was this patch tested?
Failure in https://github.com/apache-spark-on-k8s/spark/issues/617
reproduced on Kubernetes 1.6.x and 1.8.x. Built the driver image with this
patch and observed fixed
https://github.com/apache-spark-on-k8s/spark/issues/617 on Kubernetes 1.6.x.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/adelbertc/spark k8s-expose-driver-ports
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/21884.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #21884
----
commit 48d07c5578201876bdbccac7208ae868bab2e2af
Author: Adelbert Chang <adelbert.chang@...>
Date: 2018-07-26T17:42:43Z
k8s: explicitly expose ports on driver container
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
