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

[DataShare] / [ 2019-11-01]: SDK error out when the response of delete LRO is empty #839

Closed
1 task done
ziyeqf opened this issue Jan 24, 2024 · 4 comments · Fixed by hashicorp/pandora#3691 or hashicorp/terraform-provider-azurerm#24619
Assignees
Labels
base-layer/polling bug Something isn't working

Comments

@ziyeqf
Copy link

ziyeqf commented Jan 24, 2024

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

DataShare

API Versions Used

2019-11-01

Description

When Accouns_Delete is called, the service might return HTTP 202 without response body. While the SDK tried to unmarshal the empty body, which will error out.
Swagger: https://github.com/Azure/azure-rest-api-specs/blob/21a8d55d74e4425e96d76e5835f52cfc9eb95a22/specification/datashare/resource-manager/Microsoft.DataShare/stable/2019-11-01/DataShare.json#L186
SDK code:

if err = resp.Unmarshal(&result.Model); err != nil {

References

No response

@tombuildsstuff
Copy link
Contributor

@ziyeqf thanks for the bug report, taking a look into this now 👍

@tombuildsstuff tombuildsstuff added bug Something isn't working base-layer/polling labels Jan 24, 2024
@tombuildsstuff tombuildsstuff self-assigned this Jan 24, 2024
@ziyeqf
Copy link
Author

ziyeqf commented Jan 24, 2024

Thanks @tombuildsstuff, the azurerm_site_recovery_fabric resource seems to have the same issue during creation.

@tombuildsstuff
Copy link
Contributor

@ziyeqf indeed, checking the tests Machine Learning and a few others do too

@tombuildsstuff
Copy link
Contributor

#840 - will vendor that into the Provider shortly.

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