Skip to content
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

support chain of credential providers for AWS #149

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

MartinWeindel
Copy link
Member

What this PR does / why we need it:
For AWS Route53 support is added for the chain of credential providers.
If the provider Secret contains the data key AWS_USE_CREDENTIALS_CHAIN, no credentials need to be specified.
Instead the chain of credential providers is used to search for credentials in environment variables, shared credential file, and EC2 Instance Roles.

This PR is a slightly adapted variant of the PR #143.
Thanks to @zjj2wry!

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

AWS Route53: support for chain of credential providers

@gardener-robot gardener-robot added needs/review Needs review size/s Size of pull request is small (see gardener-robot robot/bots/size.py) labels Jan 5, 2021
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 5, 2021
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jan 5, 2021
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jan 14, 2021
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2021
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jan 14, 2021
@MartinWeindel
Copy link
Member Author

@zjj2wry Have you time to test this PR before we are releasing it?

@MartinWeindel MartinWeindel merged commit 8390efa into master Jan 20, 2021
@MartinWeindel MartinWeindel deleted the aws-external-creds branch January 20, 2021 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/review Needs review size/s Size of pull request is small (see gardener-robot robot/bots/size.py)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants