You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Background: Running the configured provider when access_token is a static value and credentials are set by an env variable fails, it seems the test only succeeded previously because access_token was set to an unknown resource attribute. And it fails with 0.15 because there is a new change that validates the complete configuration before configuring the provider.
In this case it seems like an explicitly-set access_token should override a defaulted credentials, but we aren't sure that assumption should be generalized for all ConflictsWith/DefaultFunc situations. So the suggestion was that maybe we leave DefaultFunc unset and handle the environment variables directly inside Configure instead.
We'll need to review the precedence of access_token and credentials after the logic is moved into Configure. This will be a breaking change so we will address it in 4.0.0
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Background: Running the configured provider when
access_token
is a static value andcredentials
are set by an env variable fails, it seems the test only succeeded previously becauseaccess_token
was set to an unknown resource attribute. And it fails with0.15
because there is a new change that validates the complete configuration before configuring the provider.In this case it seems like an explicitly-set
access_token
should override a defaultedcredentials
, but we aren't sure that assumption should be generalized for allConflictsWith
/DefaultFunc
situations. So the suggestion was that maybe we leaveDefaultFunc
unset and handle the environment variables directly inside Configure instead.We'll need to review the precedence of
access_token
andcredentials
after the logic is moved into Configure. This will be a breaking change so we will address it in4.0.0
Failed test: https://ci-oss.hashicorp.engineering/buildConfiguration/GoogleCloud_ProviderGoogleCloudGoogleProject/180134?showLog=180134_209134_208405.208416.209134&logFilter=debug
The text was updated successfully, but these errors were encountered: