azurerm_automation_schedule_resource: Changing Automation Time Helper to be 'Etc/UTC' rather than just 'UTC'

[SudoSpartanDan]

Fixes #13666

Per Microsoft Support:

Support:

Per one SME this is expected behavior and the “norm” now. Just to verify I went to another SME to see if they told me the same thing. I am currently waiting to hear back from them. I have resurfaced the conversation I had with them so I expect to hear back from them today. It would seem this is expected behavior at this point though.

Me:

Thanks for digging in. So, just to verify, we should expect the timezone to be “Etc/UTC” rather than “UTC”? Is this just for the automation API or for all APIs?

Support:

That I cannot answer because I don’t have insights into other teams API’s. For Monitoring and Automation, as of right now yes.

make acctests SERVICE='automation' TESTARGS='-run=TestAccAutomationSchedule_oneTime_basic' TESTTIMEOUT='60m'
==> Checking that code complies with gofmt requirements...
==> Checking that Custom Timeouts are used...
==> Checking that acceptance test packages are used...
TF_ACC=1 go test -v ./internal/services/automation -run=TestAccAutomationSchedule_oneTime_basic -timeout 60m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccAutomationSchedule_oneTime_basic
=== PAUSE TestAccAutomationSchedule_oneTime_basic
=== CONT  TestAccAutomationSchedule_oneTime_basic
--- PASS: TestAccAutomationSchedule_oneTime_basic (99.90s)
PASS
ok      github.com/hashicorp/terraform-provider-azurerm/internal/services/automation    99.949s

[jackofallops]

Hi @SudoSpartanDan - Thanks for this. I've left a couple comments inline about how we'd need to approach this change. Since Azure is accepting both values currently, and we need to avoid unexpected diffs for users that are relying on the current default value, we need to proceed in a staged manner. Additionally, not every service that uses timezone strings may have rolled out this change in all regions.

Thanks again.

[SudoSpartanDan]

Thanks @jackofallops good point on those. Initially, I saw that only the automation tools were using the time helper, so that's why I made the abrupt change. It's a good point, though, that others could use this as well.

[katbyte]

Thanks @SudoSpartanDan - LGTM ⛏️

[github-actions]

This functionality has been released in v2.83.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.