azurerm_recovery_services_protected_vm documentation is not consistent with code requirements

[brenak]

https://www.terraform.io/docs/providers/azurerm/r/recovery_services_protected_vm.html

The documentation states location, source_vm_name, and backup_policy_name are required. The code does not have location,source_vm_name or backup_policy_name. It does have backup_policy_id though. Code shows the following as required:

resource_group_name
recovery_vault_name (forces new)
source_vm_id (forces new)

The following as optional:
backup_policy_id (forces new)

Originally posted by @brenak in #1637 (comment)

[katbyte]

Hi @brenak,

Thanks for opening this issue, this has already been fixed in #2154 & merged and should go out in 1.18 🙂

As such i am going to close the issue.

[ghost]

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!