Skip to content

Conversation

@lucian-ioan
Copy link
Contributor

@lucian-ioan lucian-ioan commented May 30, 2024

Proposed commit message

Add overview dashboard for both logs and metrics using common Azure fields.
Add navigation panel to all dashboards.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Currently there the limitation that the user needs to create the custom data view for the dashboard to work:

custom_data_view

The ID of the Data View has to be exactly: 713624b0-5d1c-4c3a-a3a8-4a8a76c1d3b1.

I am working on adding an ad-hoc data view similar to the one in Elastic Agent Overview Dashboard before merging this.

Related issues

Screenshots

Overview:
overview

Logs:
logs

Metrics:
metrics

@lucian-ioan lucian-ioan added the Integration:azure Azure Logs label May 30, 2024
@lucian-ioan lucian-ioan self-assigned this May 30, 2024
@elasticmachine
Copy link

💚 Build Succeeded

cc @lucian-ioan

@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@lucian-ioan lucian-ioan assigned muthu-mps and unassigned muthu-mps May 30, 2024
@lucian-ioan lucian-ioan requested a review from muthu-mps May 30, 2024 00:49
@muthu-mps muthu-mps marked this pull request as ready for review May 30, 2024 06:59
@muthu-mps muthu-mps requested a review from a team as a code owner May 30, 2024 06:59
@muthu-mps
Copy link
Contributor

The datastream.dataset filter may not have any impact in the overview dashboard as we are already filtering only the openAI logs and metrics dataset in the visualisations. Lets keep it for now and we will decide on how to handle the filter in future.

Copy link
Contributor

@muthu-mps muthu-mps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The first cut looks good to me!

@muthu-mps muthu-mps merged commit fb56d46 into elastic:main May 30, 2024
@elasticmachine
Copy link

Package azure_openai - 0.5.0 containing this change is available at https://epr.elastic.co/search?package=azure_openai

@andrewkroh andrewkroh added Integration:azure_openai Azure OpenAI and removed Integration:azure Azure Logs labels Jul 22, 2024
@lucian-ioan lucian-ioan deleted the add_azure_openai_overview_dashboard branch February 6, 2025 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4 participants