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

Issue with [operationalinsights] / API Version [2021-06-01] - LRO operation does not expect 202 status code #735

Closed
1 task done
ziyeqf opened this issue Nov 13, 2023 · 2 comments

Comments

@ziyeqf
Copy link

ziyeqf commented Nov 13, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Service Used

loganalytics

API Versions Used

2021-06-01

Description

The method is only expecting HTTP 200, code link, while it's a LRO, and it will return HTTP 202. I assume it's a Swagger issue, so I opened one for that: Azure/azure-rest-api-specs#26678
But just a thought if we can add 202 to ExpectedStatusCodes for LROs when generating the sdk?

References

No response

@tombuildsstuff
Copy link
Contributor

@ziyeqf

But just a thought if we can add 202 to ExpectedStatusCodes for LROs when generating the sdk?

In this case the data needs to be updated, as you've done in this PR however we can look to add a Data Workaround for this in the interim to workaround this issue - until the upstream issue is resolved.

I've added the upstream PR to our list of open issues fwiw, so we can track that on there 👍

@tombuildsstuff
Copy link
Contributor

Fixed via hashicorp/pandora#3478

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants