-
Notifications
You must be signed in to change notification settings - Fork 29
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
Injected identity #78
Injected identity #78
Conversation
eae320f
to
5f4a080
Compare
Signed-off-by: Luis Davim <luis.davim@sendoso.com>
Signed-off-by: Luis Davim <luis.davim@sendoso.com>
Signed-off-by: Luis Davim <luis.davim@sendoso.com>
5f4a080
to
f16f78b
Compare
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 a lot for the contribution! Let's agree and fix Markdown style changes in a separate issue, please revert the unrelated changes
These were automatically done by my IDE. I'll revert them. |
This reverts commit ba13985. Signed-off-by: Luis Davim <luis.davim@sendoso.com>
684edd6
to
2e5a3ee
Compare
Done, I've reverted the commit with the linter changes, should I open a separate PR for that? |
@luisdavim yes, please. I think you have a similar linting config with @vglafirov, it popped up in his previous PRs :) So it would be great if we solve the markdown styling once and for all |
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.
@luisdavim thanks a lot for the great contribution!
Description of your changes
InjectedIdentity
was wrongly advertised as a valid credentials source for the provider config.Fixes #70
I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
I have manually tested the setup in a real EKS cluster.