Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Resync kube upstream #192

Closed
wants to merge 0 commits into from
Closed

[WIP] Resync kube upstream #192

wants to merge 0 commits into from

Conversation

ash211
Copy link

@ash211 ash211 commented May 23, 2017

NOSQUASH -- merge when merging this, don't squash and merge

Planning to add another commit or two to this before merging

@ash211 ash211 force-pushed the resync-kube-upstream branch 2 times, most recently from 86a9fc9 to f9fbf62 Compare May 26, 2017 05:58
@robert3005
Copy link

please fix #191 first

@ash211
Copy link
Author

ash211 commented Jun 13, 2017

@mccheah I'd like to have this PR pull in only stuff that's merged from upstream, and do the SBT fixes in a separate PR. Just a matter of rearranging commits later

@mccheah
Copy link

mccheah commented Jun 13, 2017

I'll rebase to remove build-related commits later.

@robert3005 robert3005 mentioned this pull request Jun 13, 2017
@robert3005
Copy link

We should add spark-kubernetes to dists boms

@mccheah
Copy link

mccheah commented Jun 13, 2017

this wasn't meant to be merged

@ash211
Copy link
Author

ash211 commented Jun 13, 2017

Wasn't merged -- I'll open a new PR when the SBT fix lands

@ash211
Copy link
Author

ash211 commented Jun 14, 2017

Replaced by #208

dvogelbacher pushed a commit that referenced this pull request Jul 30, 2018
…client (squashed) (#192)

* Allow the driver pod's credentials to be shipped through secrets.

* Fix scalastyle

* Change apiserver -> authentication

* Address comments.

Also some quality of life fixes, most notably formatting all of the
documentation strings in config.scala to no longer use triple quotes.
Triple quoted strings are difficult to format consistently.

* Fix scalastyle

* Fix comment

* Remove unnecessary constants

* Remove unnecessary whitespace

* Authentication -> Authenticate

(cherry picked from commit d5502ed)
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.

3 participants