UPDATE delete VMs that are in failed state

[HappyTobi]

Hi @tombuildsstuff,

here is just a fix to delete VM's that are in failed state.

I also deleted the comments about "power off was saving costs, thats wrong see also the azure sdk comment: power off)

The scenario for that was that you import you VM's into your TF state and some of them are at a "Failed" state. Atm the VM's couldn't be delete with TF because the "power off" will fail.

With the small fix it was possible to clean the VM's.

Bg
Tobi

[tombuildsstuff]

hi @HappyTobi

Thanks for this PR - I've taken a look through and left some comments inline but if we can fix those up then we should be able to run the tests and get this merged 👍

Thanks!

[HappyTobi]

Hi @tombuildsstuff,

I changed the comments hope you like that.

Bg

[tombuildsstuff]

LGTM - thanks for pushing those changes @HappyTobi

[tombuildsstuff]

Tests look good 👍

[ghost]

This has been released in version 2.45.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 2.45.0"
}
# ... other configuration ...

[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!