You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@libcloud.apache.org by GitBox <gi...@apache.org> on 2019/08/11 21:45:21 UTC
[GitHub] [libcloud] c-w opened a new pull request #1340: Update Azure Blob
Storage API version to 2016-05-31
c-w opened a new pull request #1340: Update Azure Blob Storage API version to 2016-05-31
URL: https://github.com/apache/libcloud/pull/1340
## Update Azure Blob Storage API version to 2016-05-31
### Description
To enable the Azure Blobs driver to upload larger files, this pull request updates the blob storage API version used by the driver to [2016-05-31](https://docs.microsoft.com/en-us/rest/api/storageservices/version-2016-05-31) which raised the size of block blobs that can be uploaded in a single request to 100MB.
To leverage the full capacity of the new storage API version, the maximum block size in the Azure Blobs driver is also increased to 100MB. A code comment from 2013 cautioned about Azure instability on requests over 4MB, however, I haven't seen any stability issues with this change during over 200 runs of the integration tests using file uploads ranging from 4MB to 100MB.
Given that the Azure Blobs driver currently does not support chunked upload, increasing the maximum upload size seems to me like a pragmatic fix to expand the driver's functionality until a more comprehensive refactor can be made to re-instate the block-list workflow.
### Status
- done, ready for review
### Checklist
- [x] Code linting ([build passed](https://travis-ci.org/CatalystCode/libcloud/builds/570564430))
- [x] Documentation ([updated changelog](https://github.com/CatalystCode/libcloud/blob/azure-blobs-api-version-update/CHANGES.rst#storage))
- [x] Tests
- [integration test 1](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=322)
- [integration test 2](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=323)
- [integration test 3](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=324)
- [integration test 4](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=325)
- [integration test 5](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=326)
- [integration test 6](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=327)
- [integration test 7](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=328)
- [integration test 8](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=329)
- [integration test 9](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=330)
- [integration test 10](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=331)
- [integration test 11](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=332)
- [integration test 12](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=333)
- [integration test 13](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=334)
- [integration test 14](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=335)
- [integration test 15](https://clewolff.visualstudio.com/libcloud-tests/_build/results?buildId=336)
- [x] ICLA ([committer](http://people.apache.org/phonebook.html?uid=clewolff))
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services