-
-
Notifications
You must be signed in to change notification settings - Fork 3
fix: allow overriding of application service account #617
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
Conversation
sbernauer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice fix, thanks and thanks for the test!
It would be nice if you could also assert the used ServiceAccount on the created Pods
|
sbernauer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Sorry, I didn't know the assertion would be this complicated...
|
I had another typo but now ... |
|
Please add a release note snippet in the form "Previously, |
Release NoteThe service account of spark applications can now be overridden with pod overrides. Previously the application service account was passed as command line argument to spark-submit |
Description
Fixes #616
Definition of Done Checklist
Author
Reviewer
Acceptance
type/deprecationlabel & add to the deprecation scheduletype/experimentallabel & add to the experimental features tracker