Skip to content
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

AzureRM Backend: support for snapshotting before changes are made #18284

Closed
tombuildsstuff opened this issue Jun 19, 2018 · 10 comments · Fixed by #24069
Closed

AzureRM Backend: support for snapshotting before changes are made #18284

tombuildsstuff opened this issue Jun 19, 2018 · 10 comments · Fixed by #24069

Comments

@tombuildsstuff
Copy link
Contributor

Moving this over from hashicorp/terraform-provider-azurerm#1098

Terraform Version

v0.10.x +

Additional Context

Thinking about this a little:

  • I don't think there's a need for an explicit versioning flag, or at least it should be defaulted to true.
  • support for Snapshotting is available in both the current Storage SDK and the new Storage SDK - however we're holding off migrating to the new Storage SDK (at least in the AzureRM Provider) until it's stabilised.

References

@pmarques
Copy link
Contributor

Hi @tombuildsstuff I'm implemented a simple version of snapshotting. Added an option to enable/disable auto snapshots (enable by default)

@pmarques

This comment has been minimized.

@mjdavies

This comment has been minimized.

@arichiardi

This comment has been minimized.

@cyrilen

This comment has been minimized.

@mildwonkey
Copy link
Contributor

Hi all, thanks for your interest!

Please react to the original issue comment with 👍, which we can and do report on during prioritization. It's more effective than individual comments, which merely generate noise for people following the issue but do not help us prioritize. Thanks!

@tombuildsstuff
Copy link
Contributor Author

tombuildsstuff commented Nov 14, 2018

@pmarques sorry for the delay here, to give an update: we're doing some work in the AzureRM Backend in the near future to support additional auth methods (and try to fix some of the open issues/enhancements) - as such I believe we'll be taking a look at this as a part of that process - but I'm unable to give a timeframe at this point, unfortunately.

Thanks!

@rohrerb

This comment has been minimized.

@rdtechie
Copy link

What's the status on this issue? Any progress?

@ghost
Copy link

ghost commented Jul 26, 2020

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.

@ghost ghost locked and limited conversation to collaborators Jul 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants