-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add a non default service account #130
Conversation
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.
The search/replace went too far, but also missing setting the serviceAccountName on the Deployment for the controller manager.
@@ -1,17 +1,406 @@ | |||
4d63.com/gochecknoglobals v0.1.0/go.mod h1:wfdC5ZjKSPr7CybKEcgJhUOgeAQW1+7WcyK8OvUilfo= |
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.
I wouldn't expect any of the go dependencies to change as part of this PR
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.
yeah, running make dist
changes it for me. This happens even on main with the latest commit. I have go 1.18.1 on my system.
@@ -0,0 +1,6 @@ | |||
--- |
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.
I would tend to put this under the manager layer, but it's a bit ambiguous as both layers need to agree on the name
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.
yeah, I was going back and forth but the rbac layer had the maximum number of references 😄 I can move it to manager folder
Codecov Report
@@ Coverage Diff @@
## main #130 +/- ##
=======================================
Coverage 69.48% 69.48%
=======================================
Files 17 17
Lines 934 934
=======================================
Hits 649 649
Misses 265 265
Partials 20 20 Continue to review full report at Codecov.
|
Pull request
What this PR does / why we need it
Adding an overlay to add annotation to service account with arn role id for AWS