Skip to content

Show traffic totals for the selected metrics range#137

Merged
arjunkomath merged 3 commits into
mainfrom
fix/service-traffic-range-totals
Jul 13, 2026
Merged

Show traffic totals for the selected metrics range#137
arjunkomath merged 3 commits into
mainfrom
fix/service-traffic-range-totals

Conversation

@arjunkomath

Copy link
Copy Markdown
Member

Summary

  • show ingress and egress summaries as total request/response body bytes for the selected metrics range
  • keep the traffic chart, tooltip, and legend expressed as throughput in B/s
  • query whole-window counter increases across proxy instances with reset-safe PromQL
  • avoid stale totals and range-label mismatches while SWR retains previous data
  • degrade individual total-query failures to unavailable values without failing other metrics

Validation

  • npx tsc --noEmit
  • npx biome check --write components/service/details/service-details-overview.tsx lib/victoria-metrics.ts tests/victoria-metrics-service-metrics.test.ts
  • npm test -- --run tests/victoria-metrics-service-metrics.test.ts (5 tests passed)
  • git diff --check

Review

Smart Review approved the query approach and identified stale-data, range-label, and test-coverage gaps; those findings are addressed in this PR.

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 3bfcee3.

@arjunkomath arjunkomath merged commit f56d54a into main Jul 13, 2026
8 checks passed
@arjunkomath arjunkomath deleted the fix/service-traffic-range-totals branch July 13, 2026 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant