4.0- removes several instances of ConfigModeAttr from automation, batch, containerinstance

[stephybun]

Community Note

• Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
• Please do not leave "+1" or "me too" comments, they generate extra noise for PR followers and do not help prioritize for review

Description

azurerm_key_vault

• Fixes a 4.0 crash

azurerm_automation_runbook

• Removes ConfigModeAttr from job_schedule

azurerm_automation_webhook

• Removes ConfigModeAttr from uri

azurerm_batch_account

• Removes ConfigModeAttr from encryption in data source

azurerm_batch_pool

• Removes ConfigModeAttr from container_registries in azurerm_batch_pool

azurerm_container_group

• Removes ConfigModeAttr from exposed_port
• Deprecated gpu and gpu_limit blocks
• Update SKU and test check used for gpu tests
• Updates various checks and configs to fix existing tests
• Increases create timeout for failing gpu tests

azurerm_container_registry

• Removes ConfigModeAttr from retention_policy by replacing the property with retention_policy_in_days in 4.0
• Removes ConfigModeAttr from trust_policy by replacing the property with trust_policy_enabled in 4.0
• Removes ConfigModeAttr from encryption in azurerm_container_group
• Deprecates encryption.enabled, retention_policy, trust_policy in azurerm_container_group
• Removes Computed on network_rule_set in 4.0
• Cleans up some of the tests for azurerm_container_group and updates the update function to use d.HasChange

Testing

Failed tests are on main

Note

If this PR changes meaningfully during the course of review please update the title and description as required.

[stephybun]

Added here until we know more about what this schema check is trying to prevent

[jackofallops]

Thanks @stephybun - LGTM 🦖