feat: retry read calls for upload sessions

deepset-ai / deepset-cloud-sdk

A Python SDK to interact with deepset Cloud

Apache License 2.0

9 stars 2 forks source link

feat: retry read calls for upload sessions #60

Closed ArzelaAscoIi closed 1 year ago

ArzelaAscoIi commented 1 year ago

Related Issues

hotfix

Proposed Changes?

we have seen that the api can sporadically return a 502 if its under load during uploading. So we should add a retry to listing the sessions and fetching the status

How did you test it?

unit tests

Checklist

[ ] I have updated the referenced issue with new insights and changes
[ ] If this is a code change, I have added unit tests
[ ] I've used the conventional commit specification for my PR title
[ ] I updated the docstrings
[ ] If this is a code change, I added meaningful logs and prepared Datadog visualizations and alerts

github-actions[bot] commented 1 year ago

Coverage report

The coverage rate went from 96% to 96% :arrow_up: The branch rate is 87%.

100% of new lines are covered.

Diff Coverage details (click to unfold)

### deepset_cloud_sdk/_api/upload_sessions.py `100%` of new lines are covered (`100%` of the complete file). ### deepset_cloud_sdk/_service/files_service.py `100%` of new lines are covered (`96%` of the complete file).