-
Notifications
You must be signed in to change notification settings - Fork 37
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
New AzCore API Breaks Environment #3795
Comments
Edit: nevermind, I see the issue and it's independent of the auth method. |
Correct. |
This issue has been addressed in PR #3799 and shipped in release v2.78.0. |
#3802 is most likely the fix. Sorry for the extra breakage. Unfortunately, we don't have usgov (or china) test infrastructure at this point, so we're relying on unit tests here, but these SDK internals are sometimes hard to cover. |
Fixes #3795, hopefully. The way Azure's azcore and azidentity SDKs are designed makes it hard to test this, but reading [the source here](https://github.com/Azure/azure-sdk-for-go/blob/sdk/azcore/v1.16.0/sdk/azcore/arm/runtime/pipeline.go#L60) it looks like this change should fix the issue.
This issue has been addressed in PR #3802 and shipped in release v2.79.0. |
What happened?
Hello,
I was previously using
v2.60.0
of the Azure Native plugin with Azure Government with success but recently decided decided to bump up to the latestv2.77.0
and noticed that all my CI builds were failing.I started investigating and found that in the
v2.71.0
release notes there is mention of a new AzCore backend for the API that has been enabled by default and it notes instructions to disable it to fall back to the previous implementation.After finding that and disabling the new AzCore backend, my builds are working again.
It appears that the new AzCore API changes are not respecting the
ARM_ENVIRONMENT
/azure-native:environment
settings as it is clearly dialing the Azure Commercial endpoints instead of the Government ones.Example
azure-native:environment
in the Pulumi config for the stackpulumi up
Output of
pulumi about
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).
The text was updated successfully, but these errors were encountered: