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

modules: update go-azure-helpers to v0.5.0 #22248

Merged
merged 1 commit into from
Aug 1, 2019

Conversation

vancluever
Copy link
Contributor

modules: update go-azure-helpers to v0.5.0

To help address the issues posed on #22087 and #22085.

Depends on #22247.

@vancluever vancluever changed the title V0.12.5 updated azure helpers modules: update go-azure-helpers to v0.5.0 Jul 30, 2019
@vancluever
Copy link
Contributor Author

Note that the updates to arm_client.go now mean that both the accounts and groups client both use the same sender; I don't think this is an issue judging from what buildSender() is actually doing. Some 👀 would be nice though.

@vancluever vancluever force-pushed the v0.12.5-updated-azure-helpers branch from 21cb499 to 00a576c Compare July 30, 2019 16:14
@pselle pselle requested a review from a team July 30, 2019 18:08
@tamalsaha
Copy link

FWIW, LGTM.

To help address the issues posed on #22087 and #22085.
@vancluever vancluever force-pushed the v0.12.5-updated-azure-helpers branch from 00a576c to d43fc71 Compare August 1, 2019 18:57
@vancluever vancluever merged commit e0329c2 into master Aug 1, 2019
radeksimko added a commit that referenced this pull request Aug 20, 2019
This is to allow Terraform providers to upgrade to at least
one more minor version of the plugin SDK without major UX hiccups.

This concludes (unsuccessful) experiments involving upgrades
to SDK with Azure/go-autorest#455

Even with that patch all providers still experience broken UX
as described in #22490

This downgrade reduces the uncomfort to only a handful of providers
from >100s. The affected providers more or less directly depend on
Azure SDK(s), which is ~8.

Affected providers practically cannot consume Terraform Plugin SDK
with this patch (downgraded Azure SDKs) and can just wait for
extracted Terraform Plugin SDK which is planned to be released soon.

This reverts the following PRs:

 - #22247
 - #22248
 - #22524
 - #22525

and it is otherwise result of the following commands

```
go get github.com/Azure/azure-sdk-for-go@v21.3.0
go get github.com/hashicorp/go-azure-helpers@166dfd221bb2
go mod tidy
```
@ghost
Copy link

ghost commented Sep 1, 2019

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.

@ghost ghost locked and limited conversation to collaborators Sep 1, 2019
@pselle pselle deleted the v0.12.5-updated-azure-helpers branch November 13, 2019 19:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants