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.

Sign up for GitHub

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

Jump to bottom

[BlobStorage] Unsafe retry on 500 OperationTimedOut error #39615

Open
annatisch opened this issue Feb 6, 2025 · 1 comment
Open

[BlobStorage] Unsafe retry on 500 OperationTimedOut error #39615

annatisch opened this issue Feb 6, 2025 · 1 comment
Labels
Client This issue points to a problem in the data-plane of the library. Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files)

Comments

@annatisch
Copy link
Member

  • Package Name: azure-storage-blob (and probably other storage packages)
  • Package Version: latest
  • Operating System: Any
  • Python Version: Any

Describe the bug
The Python SDK will retry a 500 status with error code OperationTimedOut.
https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/storage/azure-storage-blob/azure/storage/blob/_shared/policies.py#L87-L93

According to the REST API docs, this retry is unsafe:
https://learn.microsoft.com/en-us/rest/api/storageservices/common-rest-api-error-codes

This issue can result in data corruption:
Azure/azure-sdk-for-go#24027
@annatisch annatisch added the Storage Storage Service (Queues, Blobs, Files) label Feb 6, 2025
@kashifkhan kashifkhan added the Service Attention Workflow: This issue is responsible by Azure service team. label Feb 6, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 6, 2025

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @jalauzon-msft @vincenttran-msft.

@kristapratico kristapratico added the Client This issue points to a problem in the data-plane of the library. label Feb 7, 2025
@vincenttran-msft vincenttran-msft pinned this issue Feb 10, 2025
@vincenttran-msft vincenttran-msft unpinned this issue Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Client This issue points to a problem in the data-plane of the library. Service Attention Workflow: This issue is responsible by Azure service team. Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kashifkhan @annatisch @kristapratico