Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[O11y][Azure Metrics] Migrate Blob Storage Overview dashboard to lens #9503

Merged

Conversation

harnish-elastic
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-elastic harnish-elastic added the Integration:azure_metrics Azure Resource Metrics label Apr 3, 2024
@harnish-elastic harnish-elastic self-assigned this Apr 3, 2024
@harnish-elastic harnish-elastic marked this pull request as ready for review April 3, 2024 10:14
@harnish-elastic harnish-elastic 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

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-elastic harnish-elastic merged commit 77cf428 into elastic:main Apr 10, 2024
5 checks passed
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants