-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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.
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
Azure backend: support snapshots/versioning #24069
Azure backend: support snapshots/versioning #24069
Conversation
Just mess up my state. @tombuildsstuff please, don't let this MR rotten in 2020. If you have not enough time, who can help out? |
Really looking forward to this! |
Hi @tombuildsstuff , any chance we can get this merged any time soon? We urgently need this in order to prevent incidents like the one @WebSpider experienced. Thanks! |
Really looking forward to this, any news on when it will be released? |
@tombuildsstuff I just messed my state, thought I was just going to restore a previous version & came to find out that versioning is not only not enabled by default but not even possible. This would have likely saved me a few hours of work |
Any update on this one please? I can see that review requests are pending From Feb 10th, which is > 2 months ago now. |
@tombuildsstuff, Could you please help to review this RP? It's pending for about 3 months. Thanks for your help in advance. |
Dependent on #24669 |
Codecov Report
|
Co-authored-by: Reda Ahdjoudj <[email protected]> Co-authored-by: Patrick F. Marques <[email protected]>
31c8fff
to
b9584d7
Compare
I have squashed the commits and rebased after #24669 was merged in. Could you please review @tombuildsstuff, especially since you are very familiar with Giovanni ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hey @evenh
Thanks for this PR - apologies for the delay reviewing this - as mentioned previously we've been waiting to move off the deprecated Storage SDK in the Azure SDK for Go in favour of Giovanni.
Taking a look through this mostly LGTM - the main issue being switching to use the terminology that Azure uses instead of what Amazon uses ("snapshot" instead of "versioning") - but if we can fix that up then we should be able to take another look and kick off the tests :)
Thanks!
Hi there, just wanted to let you know that if you enable soft delete on the storage account, you actually get new snapshots of the state every time you run terraform. See this for details: https://docs.microsoft.com/en-us/azure/storage/blobs/soft-delete-overview (Put operation) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thanks for this @evenh
Ran the tests for this and they pass - thanks for this @evenh / sorry for the delay getting this merged 👍
|
Hooray, thanks :-) |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Rebased PR with some small fixes. Originally by @pmarques (2018) and @rahdjoudj (2019).
Fixes #18284
Closes #18512
Closes #21888