Skip to content

Conversation

ken-schneider
Copy link
Contributor

What does this PR do?

Updates the Versa integration's OOTB Dashboard to fix the dashboard name as well as update the QoS metric prefixes. Additionally, this updates the metadata.csv to include new metrics exposed by the integration.

Motivation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Copy link

github-actions bot commented Sep 4, 2025

⚠️ Recommendation: Add qa/skip-qa Label

This PR does not modify any files shipped with the agent.

To help streamline the release process, please consider adding the qa/skip-qa label if these changes do not require QA testing.

hestonhoffman
hestonhoffman previously approved these changes Sep 4, 2025
@temporal-github-worker-1 temporal-github-worker-1 bot dismissed hestonhoffman’s stale review September 5, 2025 18:37

Review from hestonhoffman is dismissed. Related teams and files:

  • documentation
    • versa/assets/dashboards/versa.json
@ken-schneider ken-schneider added this pull request to the merge queue Sep 10, 2025
Merged via the queue into master with commit 87aab10 Sep 10, 2025
46 checks passed
@ken-schneider ken-schneider deleted the ken/update-versa-integration branch September 10, 2025 13:59
datadog-agent-integrations-bot bot pushed a commit that referenced this pull request Sep 10, 2025
* fix OOTB dashboard name

* fix dashboard qos metric prefixes

* add link speed/utilization metrics

* remove invalid cloned comment

* update dashboard ordering, maximum utilization calculations

* use avg for site metrics

(cherry picked from commit 87aab10)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants