Skip to content

Conversation

@harnish-crest-data
Copy link
Contributor

  • Enhancement

Proposed commit message

  • Migration of the visualizations to lens.
  • Please refer the Screenshots mentioned in the description of the PR for comparison.

Migration stats

Azure Metrics Before Migration   After Migration  
  Lens Visualization Lens Visualization
[Azure Metrics] Blob Storage Overview 0 10 10 0

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.

Related issues

Screenshots

Before

image

After

image

@harnish-crest-data harnish-crest-data added the Integration:azure_metrics Azure Resource Metrics label Apr 3, 2024
@harnish-crest-data harnish-crest-data self-assigned this Apr 3, 2024
@harnish-crest-data harnish-crest-data marked this pull request as ready for review April 3, 2024 10:14
@harnish-crest-data harnish-crest-data requested a review from a team as a code owner April 3, 2024 10:14
@elasticmachine
Copy link

🚀 Benchmarks report

Package azure_metrics 👍(3) 💚(0) 💔(1)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
compute_vm_scaleset 250000 200000 -50000 (-20%) 💔

To see the full report comment with /test benchmark fullreport

@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No Coverage information No data about Coverage
No Duplication information No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link

💚 Build Succeeded

cc @harnish-elastic

Copy link
Contributor

@milan-elastic milan-elastic left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@zmoog zmoog left a comment

Choose a reason for hiding this comment

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

LGTM

@harnish-crest-data harnish-crest-data merged commit 77cf428 into elastic:main Apr 10, 2024
@elasticmachine
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:azure_metrics Azure Resource Metrics

4 participants