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

sdk/client/resourcemanager: supporting 202's with a self reference #597

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

tombuildsstuff
Copy link
Contributor

Most of API Management returns a 202 with a self reference containing an asynctoken, as such if a self-reference exists then we can use this URI directly.

Fixes:

Error: creating/updating Api (Subscription: "*******"
        Resource Group Name: "acctestRG-230803055317928116"
        Service Name: "acctestAM-230803055317928116"
        Api: "acctestAMA-230803055317928116"): performing CreateOrUpdate: no applicable pollers were found for the response

From hashicorp/terraform-provider-azurerm#22783

Most of API Management returns a 202 with a self reference containing an `asynctoken`, as such if a self-reference
exists then we can use this URI directly.

Fixes:

```
Error: creating/updating Api (Subscription: "*******"
        Resource Group Name: "acctestRG-230803055317928116"
        Service Name: "acctestAM-230803055317928116"
        Api: "acctestAMA-230803055317928116"): performing CreateOrUpdate: no applicable pollers were found for the response
```

From hashicorp/terraform-provider-azurerm#22783
Copy link
Contributor

@manicminer manicminer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, LGTM 👍

@tombuildsstuff tombuildsstuff added the release-once-merged The SDK should be released once this PR is merged label Aug 3, 2023
@tombuildsstuff tombuildsstuff merged commit a12e77e into main Aug 3, 2023
5 checks passed
@tombuildsstuff tombuildsstuff deleted the b/poller-apimanagement branch August 3, 2023 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base-layer/polling release-once-merged The SDK should be released once this PR is merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants