-
Notifications
You must be signed in to change notification settings - Fork 47
fix: adding dynamic threshold of link speed in 2 network tables #4062
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
Conversation
# Conflicts: # docs/ontap-metrics.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates network monitoring dashboards to use dynamic percentage-based thresholds instead of fixed absolute byte values for link speed monitoring. The change affects two network tables (NIC ports and Link Aggregation Groups) in the Grafana dashboard by introducing new percentage metrics (nic_ifgrp_rx_perc, nic_ifgrp_tx_perc, nic_rx_percent, nic_tx_percent) and updating the visualization thresholds to use percentage mode with configurable warning levels (50%, 75%, 90%).
Key changes:
- Added new percentage-based metrics for network interface group receive/transmit bandwidth utilization
- Updated Grafana dashboard to display bandwidth usage as percentages with dynamic thresholds
- Modified backend collectors to calculate and populate percentage metrics for port aggregation groups
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| mcp/metadata/ontap_metrics.json | Added metric definitions for nic_ifgrp_rx_perc and nic_ifgrp_tx_perc |
| grafana/dashboards/cmode/network.json | Updated dashboard configuration to include percentage-based visualizations with dynamic thresholds, added new query references for percentage metrics, and reorganized column ordering |
| docs/ontap-metrics.md | Added documentation for the new percentage metrics including API endpoints and Grafana dashboard references |
| cmd/tools/generate/counter.yaml | Added counter definitions for nic_ifgrp_rx_perc and nic_ifgrp_tx_perc metrics |
| cmd/collectors/zapiperf/plugins/nic/nic.go | Extended PortData population to include percentage values for read/write operations |
| cmd/collectors/restperf/plugins/nic/nic.go | Extended PortData population to include percentage values for read/write operations |
| cmd/collectors/commonutils.go | Added ReadPerc and WritePerc fields to PortData struct and updated aggregation logic to sum percentage values |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


No description provided.