libcloud-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [libcloud] c-w opened a new pull request #1340: Update Azure Blob Storage API version to 2016-05-31
Date Sun, 11 Aug 2019 21:45:21 GMT
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

Mime
View raw message