Skip to content

add value to let you annotate the service account#6586

Closed
timothy-spencer wants to merge 5 commits intogravitational:masterfrom
timothy-spencer:annotateServiceAccount
Closed

add value to let you annotate the service account#6586
timothy-spencer wants to merge 5 commits intogravitational:masterfrom
timothy-spencer:annotateServiceAccount

Conversation

@timothy-spencer
Copy link
Copy Markdown
Contributor

For similar reasons to #5271, I would like to be able to annotate the serviceaccount.

Let me know if you like this or have suggestions, whatever! Thanks, and have fun!

@klizhentas
Copy link
Copy Markdown
Contributor

@timothy-spencer thank you for your PR. We will review and get back to you! Have a great day 👋

@russjones
Copy link
Copy Markdown
Contributor

@timothy-spencer Thank you for your contribution!

@webvictim You're reworking Helm charts in #6344, can you take a look at this and see if these changes make sense in the context of new Helm charts?

@russjones russjones requested a review from webvictim April 28, 2021 21:17
@webvictim
Copy link
Copy Markdown
Contributor

webvictim commented Apr 29, 2021

@timothy-spencer I'll add support for serviceAccount annotations into #6344, as that PR is introducing a whole annotations section and it'll save refactoring it twice.

Edit: added in d3eb086

@timothy-spencer
Copy link
Copy Markdown
Contributor Author

Supercool. That looks great to me! Thank you! This will make it much easier to integrate with EKS.

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