-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Feature Request: azurerm_recovery_services_vault vm enrollment #1143
Comments
Some initial investigation into this - based on this ARM Template:
It appears this is available through the Protected Items API: https://docs.microsoft.com/en-us/rest/api/backup/protecteditems and appears to be in the Go SDK: https://github.com/Azure/azure-sdk-for-go/blob/master/services/recoveryservices/mgmt/2017-07-01/backup/protecteditemsgroup.go#L49 |
Hi @tombuildsstuff, Re your reply to #1007, your comment above appears to only address the capability to Backup a VM, whereas I am looking for Disaster Recovery (Site Recovery) support. |
Has any progress been made on adding the VM Backups, and backup policies as suggested in this request? thanks |
@philipstreet-hiscox apologies - missed this reply: would you mind opening a separate Issue for that feature request specifically, with an example if you've got one? This would help us when writing the test-cases for it :) @DamianFlynn this is on our roadmap for the near future - but I can't give a specific date at this time. Since you're asking for them - would you be able to give an example of Backup Policies that we could use as a basis for this? Thanks! |
@tombuildsstuff Hi Tom, I did open a feature request for this (#1007) but you closed it saying that it was covered by another feature request, but it wasn't. Can we open my original request rather than creating a new feature request? Phil |
@philipstreet-hiscox after taking another look into this, I'll re-open the other issue; apologies for the confusion here. I'll add some additional context to the other issue & rename the title to match. Thanks! |
@tombuildsstuff Many thanks Tom. I can provide specific scenario details, which I'll add to the other issue. |
I am also very interested in this feature. My use case is just to add created VMs to preconfigured Recovery Services. This is crucial to maintain automated configuration and deployment. 3 months passed, any updates on ETA? |
Attempted workaround using Azure cli embedded in local-exec also runs into issues because Terraform's "on-destroy" is broken in providers. I've tried this:
But the on destroy clause is ignored, which means the solution is not adequate. Note: The above bloc is in a module wrapping a virtual machine resource:
|
This was just released in |
Hello Tom, Katbyte et al, dont suppose you have a modifed milestone for backup? |
@phosphre VM Backups have been available since v1.17.0 (so should be available today) Since this issue's been closed for a while I'm going to lock it - however if you're looking support for a different backup type (or have questions) please feel free to open a new issue and we'll take a look :) Thanks! |
Hi there,
Terraform Version
Terraform v0.11.7
Affected Resource(s)
Please list the resources as a list, for example:
Expected Behavior
With the upcoming support for creating Recovery Service Vaults, it would be helpful to enroll any vm's that are created in the same
apply
with the Vault for backups. This would require additional work such as creating a backup policy under theazurerm_recovery_services_vault
as shown below.My current workflow is to deploy vm's with terraform and then run an additional powershell script locally that will...
It would greatly improve the workflow if instead of the above workflow, I could doing something such as...
I would be interested in working on this but have no experience in go or how the azure provider leverages azure to perform tasks. If there is official docs on the provider, I am happy to dig in.
References
Are there any other GitHub issues (open or closed) or Pull Requests that should be linked here? For example:
#995
The text was updated successfully, but these errors were encountered: