azurerm_kubernetes_cluster - support for the azure_monitor_kubernetes_metrics property

[ms-henglu]

=== RUN   TestAccKubernetesCluster_azureMonitorProfile
=== PAUSE TestAccKubernetesCluster_azureMonitorProfile
=== CONT  TestAccKubernetesCluster_azureMonitorProfile
--- PASS: TestAccKubernetesCluster_azureMonitorProfile (999.95s)
PASS

[ms-henglu]

Hi @tombuildsstuff - Thanks for the code review. I've updated this PR as suggested. Please take another look, thanks!

[stephybun]

azure and kubernetes seem redundant here, perhaps this would be sufficient

Suggested change

	"azure_monitor_kubernetes_metrics": {
	Type: pluginsdk.TypeList,
	MaxItems: 1,
	Optional: true,
	Elem: &pluginsdk.Resource{
	Schema: map[string]*pluginsdk.Schema{
	"metric_annotations_allow_list": {
	Type: pluginsdk.TypeString,
	Optional: true,
	ValidateFunc: validation.StringIsNotEmpty,
	},
	"metric_labels_allow_list": {
	Type: pluginsdk.TypeString,
	Optional: true,
	ValidateFunc: validation.StringIsNotEmpty,
	},
	},
	},
	},
	"monitor_metrics": {
	Type: pluginsdk.TypeList,
	MaxItems: 1,
	Optional: true,
	Elem: &pluginsdk.Resource{
	Schema: map[string]*pluginsdk.Schema{
	"annotations_allowed": {
	Type: pluginsdk.TypeString,
	Optional: true,
	ValidateFunc: validation.StringIsNotEmpty,
	},
	"labels_allowed": {
	Type: pluginsdk.TypeString,
	Optional: true,
	ValidateFunc: validation.StringIsNotEmpty,
	},
	},
	},
	},

[stephybun]

Thanks @ms-henglu LGTM 👍

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