Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

fix(azure): use regular storage instead of premium #694

Merged
merged 1 commit into from
Jan 10, 2017

Conversation

bacongobbler
Copy link
Member

@bacongobbler bacongobbler commented Jan 10, 2017

premium storage does not support block blobs, whereas regular storage does.

Error coming from wal-e, which makes workflow completely unusable with this setup:

azure.storage._http.HTTPError: Block blobs are not supported.

cc @dtzar @slack.

closes #693.

@deis-bot
Copy link

@dtzar and @slack are potential reviewers of this pull request based on my analysis of git blame information. Thanks @bacongobbler!

@bacongobbler
Copy link
Member Author

bacongobbler commented Jan 10, 2017

Commit that originally changed this behaviour would be bb54bef

@dtzar
Copy link
Contributor

dtzar commented Jan 10, 2017

Yes, it is true that premium storage doesn't support page blobs yet :(. https://docs.microsoft.com/en-us/rest/api/storageservices/fileservices/using-blob-service-operations-with-azure-premium-storage

We need to switch the --sku from Premium_LRS to Standard_LRS as the --sku switch is required. https://docs.microsoft.com/en-us/cli/azure/storage/account

@slack
Copy link
Member

slack commented Jan 10, 2017

Looks like Standard_LRS would work!?

https://docs.microsoft.com/en-us/rest/api/storagerp/srp_sku_types

premium storage does not support block blobs, whereas regular storage does.

Error coming from wal-e:

```
azure.storage._http.HTTPError: Block blobs are not supported.
```
@bacongobbler
Copy link
Member Author

thanks guys! Edited the PR to just switch the --sku flag to Standard_LRS.

@slack slack added the LGTM1 label Jan 10, 2017
@dtzar
Copy link
Contributor

dtzar commented Jan 10, 2017

LGTM2

@bacongobbler bacongobbler merged commit 954f425 into deis:master Jan 10, 2017
@bacongobbler bacongobbler deleted the fix-azure-docs branch January 10, 2017 22:52
@jalberto
Copy link

confirmed, it works with Standar_LRS

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

deis-database backup not working in new Azure installation causing DB to not work at all
5 participants