Skip to content

Conversation

@liyinan926
Copy link
Member

No description provided.

@foxish
Copy link
Member

foxish commented Oct 19, 2017

LGTM

@foxish
Copy link
Member

foxish commented Oct 19, 2017

Might also be good to specify kube-dns as a requirement here?
cc/ @kimoonkim

@liyinan926
Copy link
Member Author

Good point. kube-dns is required for the headless service for the driver to work.

@kimoonkim
Copy link
Member

+1

@liyinan926
Copy link
Member Author

Done.

@foxish foxish merged commit 44b1396 into apache-spark-on-k8s:master Oct 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants