Skip to content

Conversation

@mccheah
Copy link

@mccheah mccheah commented Mar 16, 2017

Closes #133

…#180)

* Allow providing an OAuth token for authenticating against k8s

* Organize imports

* Fix style

* Remove extra newline

* Use OAuth token data instead of a file.

(cherry picked from commit 1aba361)
@mccheah
Copy link
Author

mccheah commented Mar 16, 2017

Still missing 2 of the 3 PRs that we need.

foxish and others added 2 commits March 17, 2017 17:10
* Adding official alpha docker image to docs

* Reorder sections and create a specific one for "advanced"

* Provide limitations and instructions about running on GKE

* Fix title of advanced section: submission

* Improved section on running in the cloud

* Update versioning

* Address comments

* Address comments

(cherry picked from commit e5da90d)
…client (squashed) (apache#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
@mccheah
Copy link
Author

mccheah commented Mar 18, 2017

I cherry-picked apache-spark-on-k8s#192, also taking apache-spark-on-k8s#176 to avoid merge conflicts. Still need apache-spark-on-k8s#173

@ash211
Copy link

ash211 commented Mar 20, 2017

Wrong branch -- we're releasing k8s stuff off of palantir:master now. I'll set up a new PR that resyncs from k8s upstream

@ash211 ash211 closed this Mar 20, 2017
@ash211
Copy link

ash211 commented Mar 20, 2017

Replaced with #137

mccheah pushed a commit that referenced this pull request Apr 27, 2017
* Configure unit test build while banning flaky tests

* Clean up comment
dvogelbacher pushed a commit that referenced this pull request Jul 30, 2018
* Configure unit test build while banning flaky tests

* Clean up comment

(cherry picked from commit 0a22b35)
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