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

Fix #113 Allow setting of a AWS profile when creating the SDK config #126

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pwhittlesea
Copy link

@pwhittlesea pwhittlesea commented Jul 21, 2023

Fixes #113

When working with many AWS accounts you need to be able to specify different profiles when connecting to given services.

Currently this plugin does not allow the setting of a profile which means you need to remember to run Terraform with AWS_PROFILE=customprofile terraform plan or the GetClusterCredentials call will fail.

If you are using the AWS provider, running Terraform with the AWS_PROFILE overrides any explicitly defined profile on the provider.
This means when using profiles, under some conditions, you cannot use this provider and the AWS provider in the same project.

This PR adds the option to define a profile which is completely optional. If no profile is defined the provider operates in the same way as it did before.

@pwhittlesea
Copy link
Author

Currently I haven't added any tests as I wanted to see if the existing tests would still pass before I commit more time to this. Can a maintainer authorize the workflow please?

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.

Add support for setting profile like in the aws provider
1 participant