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: support for LROs which also return immediately #583

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

tombuildsstuff
Copy link
Contributor

Both API Management/Schema @ 2021-08-01/2022-08-01 and Automation/DSCConfiguration@2020-01-13-preview support returning both a LRO and an immediate response. In this case since we retrieve the original URI and get a 200 OK /without/ a provisioningState header - we'll special-case this for now.

Fixes #542

…ately

Both API Management/Schema @ 2021-08-01/2022-08-01 and Automation/DSCConfiguration@2020-01-13-preview
support returning both a LRO and an immediate response. In this case since we retrieve the original
URI and get a 200 OK /without/ a provisioningState header - we'll special-case this for now.

Fixes #542
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.

Thanks @tombuildsstuff, LGTM 🏟️

@tombuildsstuff tombuildsstuff merged commit 31cc039 into main Aug 1, 2023
5 checks passed
@tombuildsstuff tombuildsstuff deleted the b/api-management-polling branch August 1, 2023 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

automation/dscconfiguration: polling error expected either provisioningState or status
2 participants