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

preconfigureCallback: Allow Env Vars for Credentials Validation #2148

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

stack72
Copy link
Contributor

@stack72 stack72 commented Sep 20, 2022

In order to ensure that the user doesn't need to add anything to
state to skip a credentials validation or to skip the metadata api
check, we now allow a user to set the value via environment variables.

The order of operations is that we check for provider configuration
first and if we can't find it, we check for the environment variable

Therefore, a person who wants to skip the credentials validation can
now set:

AWS_SKIP_CREDENTIALS_VALIDATION=true when running pulumi up

In order to ensure that the user doesn't need to add anything to
state to skip a credentials validation or to skip the metadata api
check, we now allow a user to set the value via environment variables.

The order of operations is that we check for provider configuration
first and if we can't find it, we check for the environment variable

Therefore, a person who wants to skip the credentials validation can
now set:

AWS_SKIP_CREDENTIALS_VALIDATION=true when running pulumi up
@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@viveklak viveklak merged commit de8871b into master Oct 4, 2022
@pulumi-bot pulumi-bot deleted the add-env-vars-for-credentials branch October 4, 2022 17:28
@stack72
Copy link
Contributor Author

stack72 commented Oct 4, 2022

@viveklak this pr opens a fix for pulumi/pulumi-kubernetes-operator#291

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.

2 participants