reboot a VM only on managed disk resize

[shurikk]

related to #6063

Fixes #6063

[tombuildsstuff]

hey @shurikk

Thanks for this PR :)

Taking a look through this mostly LGTM - if we can fix up the documentation issue (and the tests pass) then this otherwise LGTM 👍

Thanks!

[tombuildsstuff]

LGTM - thanks for pushing those changes @shurikk

[tombuildsstuff]

hey @shurikk

Thanks for this PR - I've spent a little time digging into this and it appears there's some more edge-cases here - as such I hope you don't mind but I'm going to push a couple of commits to rebase this PR (to include the changes from #6207) and then handle those edge-cases so that we can get this merged.

Thanks!

[jackofallops]

LGTM 👍

[shurikk]

thanks 👍

[tombuildsstuff]

Ignoring one test failing due to capacity issues in Azure - the tests pass:

[shurikk]

any chance this fix goes into 1.xx branch? I know that 1.44 was the last one in 1.xx, this bug is somewhat dangerous.

[ghost]

This has been released in version 2.3.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.3.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 👉 hashibot-feedback@hashicorp.com. Thanks!