Closed EppO closed 1 year ago
@catriona-m I think this issue got closed accidentally, this functionality is still missing and the linked pull request doesn’t touch any Kubernetes components that I can see.
@cpressland: you're right, #21598 implemented #18809, not this one
@EppO @cpressland Although this feature is marketed as being an AKS add-on, enabling Prometheus metric collection is a combination of a separate group of resources (azurerm_monitor_data_collection_endpoint
,azurerm_monitor_data_collection_rule
and azurerm_monitor_data_collection_rule_association
). Best example of that is found here, a MS example.
As such adding this 'add-on' to the azurerm_kubernetes_cluster
will probably never happen, unless the implementation at MS side is abstracted into the AKS APIs.
The current implementation of Prometheus and AKS linking can be enabled with azurerm
via azurerm_monitor_data_collection_endpoint
,azurerm_monitor_data_collection_rule
and azurerm_monitor_data_collection_rule_association
, as such I'll close this issue for now.
The prometheusRuleGroups
resource to configure the Prometheus rules on the azurerm_monitor_workspace
is already being implemented in this PR: #21751.
Please let me know if you have any questions, I'd be happy to help!
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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Is there an existing issue for this?
Community Note
Description
Currently, there is no way to bind an existing Azure Monitor Workspace Id to an AKS cluster. The exisiting implementation (#19530 ) relies on a default Azure Monitor Workspace. This feature request is the first step to support custom Azure Monitor Workspace. Once #18809 is implemented, we could pass Azure Monitor Workspace resource or datasource to that new attribute.
New or Affected Resource(s)/Data Source(s)
azurerm_kubernetes_cluster
Potential Terraform Configuration
References