azurerm_disk_encryption_set support for enable_auto_key_rotation

[Tbohunek]

Azure Disk Encryption Set now supports automatic Key rotation

I worked by #13717 and #8670.

Did I miss anything?

[stephybun]

Hi @Tbohunek, thanks for this PR. Overall this looks good, there is one suggestion on the schema and the formatting needs to be fixed so the remaining checks can run.

[stephybun]

Could we rename this to follow our convention for boolean properties

Suggested change

	"enable_auto_key_rotation": {
	"auto_key_rotation_enabled": {

[Tbohunek]

I've renamed the variable, but I cannot format the code from the web editor. Could you please do that for me @stephybun?

[stephybun]

@Tbohunek thanks for renaming that property 🙂. The linting errors need to be addressed next before we can merge this.

[Tbohunek]

Hi @stephybun @myc2h6o I have done what I could hoping to save someone some time, but my ability to code this myself ends here. Please feel free to commit directly what you think is necessary. I will happily observe and keep that in mind for next time.

[myc2h6o]

Thanks @Tbohunek for making the pull request. I'll update the pr to resolve the comments from stephybun and ms-henglu

[Tbohunek]

@myc2h6o I just committed what I see as the only difference from your PR. @stephybun could you pls rerun actions?
If this is correct then I still don't see how to fix data though.

[myc2h6o]

Yeah saw your new change @Tbohunek , there is some naming error, e.g. rotationToLatestKeyVersionEnabled vs RotationToLatestKeyVersionEnabled I don't have permission to commit to your branch directly thus making a pr to your branch, can you take a look and merge if it looks good?
Tbohunek#1

[myc2h6o]

Forgot to run terrafmt, here is the fmt update Tbohunek#2
The acc tests _update are failing because update of auto_key_rotation_enabled requires a key vault permission but it's not added to the test case yet, I'm looking at updating it

[myc2h6o]

@Tbohunek I updated the test here: Tbohunek#3
Can you take a look? I've put the change detail and the test result there as well

[stephybun]

Thanks @Tbohunek and @myc2h6o. Tests are passing, I have one last suggestion, then this is good to go.

[stephybun]

LGTM 🚀

[github-actions]

This functionality has been released in v2.82.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.