Skip to content
This repository has been archived by the owner on Jan 9, 2020. It is now read-only.

Allow opening additional ports on the driver #78

Closed
mccheah opened this issue Feb 2, 2017 · 4 comments
Closed

Allow opening additional ports on the driver #78

mccheah opened this issue Feb 2, 2017 · 4 comments

Comments

@mccheah
Copy link

mccheah commented Feb 2, 2017

A use case @ash211 and I are building requires exposing additional ports on the driver pod. It should be straightforward enough to support either <name>=<value> ports or just a comma-separated list of numbers. I'm inclined to provide just the list of numbers to keep things simple.

We may not have to go to the extent of exposing these ports on the service we create for the Spark UI just yet - the user can currently provide their own custom label that locates their pod and create a service overlay on top of the opened ports via this configuration.

@mccheah
Copy link
Author

mccheah commented Feb 2, 2017

We should be careful to minimize the number of additions of these kinds of features - #38 is the right catch-all if the feature set grows too large.

@foxish
Copy link
Member

foxish commented Feb 2, 2017

How common a use case is it to need additional ports exposed? That should be the deciding factor for whether we want to support it as an option, or in the custom pod yamls. @ssuchter thoughts?

@ash211
Copy link

ash211 commented Feb 2, 2017

This is very common with our Spark jobs (and the path forward for all new Spark jobs we write), though I can't say how common it is elsewhere.

@mccheah
Copy link
Author

mccheah commented Feb 13, 2017

We didn't end up needing this - can revisit later if it would be convenient in general.

@mccheah mccheah closed this as completed Feb 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants