Skip to content

[O11y][Azure Metrics] Update visualizations and remove suffix #9502

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Apr 12, 2024
Merged

[O11y][Azure Metrics] Update visualizations and remove suffix #9502

merged 3 commits into from
Apr 12, 2024

Conversation

harnish-crest-data
Copy link
Contributor

  • Enhancement

Proposed commit message

  • Updated visualizations of Queue Storage Overview , Table Storage Overview , File Storage Overview & Storage Overview.

  • Updated intervals to auto wherever applicable.

  • Updated the aggregation from Average to Last Value wherever applicable.

  • Removed suffix [Azure Metrics] wherever found

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
Update visualizations of Queue Storage Overview, Table Storage Overvi…
7faff07 
…ew, File Storage Overview & Storage Overview dashboards
@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
@elasticmachine
Copy link

elasticmachine commented Apr 3, 2024 •
edited
Loading

🚀 Benchmarks report

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

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
container_service 250000 166666.67 -83333.33 (-33.33%) 💔

To see the full report comment with /test benchmark fullreport

@harnish-crest-data harnish-crest-data marked this pull request as ready for review April 3, 2024 10:15
@harnish-crest-data harnish-crest-data requested a review from a team as a code owner April 3, 2024 10:15
ali786XI
ali786XI approved these changes Apr 3, 2024
View reviewed changes
Copy link
Contributor

@ali786XI ali786XI left a comment

Choose a reason for hiding this comment

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

LGTM

zmoog
zmoog approved these changes Apr 8, 2024
View reviewed changes
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
Merge branch 'main' of https://github.com/harnish-elastic/integrations …
8a270eb 
…into azure-metrics-storage-dashboard

Conflicts:
	packages/azure_metrics/changelog.yml
	packages/azure_metrics/manifest.yml
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @harnish-elastic

@elastic-sonarqube 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

@harnish-crest-data harnish-crest-data merged commit e30c0d7 into elastic:main Apr 12, 2024
@elasticmachine
Copy link

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

@zmoog zmoog mentioned this pull request Sep 13, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zmoog zmoog zmoog approved these changes

+1 more reviewer

@ali786XI ali786XI ali786XI approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@harnish-crest-data harnish-crest-data

Labels
Integration:azure_metrics Azure Resource Metrics
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@harnish-crest-data @elasticmachine @zmoog @ali786XI