Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

WiP: diagnostic setting: metrics and logs to LogAnalytics #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zioproto
Copy link
Collaborator

Azure OpenAI has diagnostic settings that make possible to send logs and metrics to a destination.

The possible destinations are:

  • Log Analytics workspace
  • Storage account
  • Event Hub
  • partner solutions

This is how the config looks in the portal:

Screenshot 2023-11-15 at 09 07 55

I am not sure if the right approach is to have the azurerm_monitor_diagnostic_setting resource inside or outside this module.

  • Inside: the logic is quite complex because of the activation of individual logs and metrics in combination to the activation of possibly multiple destinations. Moreover there could be multiple diagnostic setting resources. See how many "NOTE:" section the azurerm_monitor_diagnostic_setting resource has.
    We could decide to support only a subset of the possible configurations.
  • Outside: requires a new output for azurerm_cognitive_account.this.id and we leave to the user the task of managing the complexity.

I am looking for feedback on how to move forward. Thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant