Skip to content

Conversation

@landongrindheim
Copy link
Member

🖼️ Context

Our custom metrics usage is currently almost entirely consumed by the high-cardinality values of http.url_details.path, which maps to over 800k values. This change is intended to normalize the paths so that we evaluate paths in a way similar to how the config/routes.rb file defines them.

How It's Used

sidecar_nginx.response_bytes_written accounts for 99.9% of our custom metrics 👇

image

This metric uses the following configuration, in which you can observe that over the past eight hours there are more than one million values mapped to its tags. 👇

image

By normalizing the the paths, we will maintain insight into general access patterns while taming our use of custom metrics.

Our custom metrics usage is currently almost entirely consumed by the
high-cardinality values of http.url_details.path, which maps to over
800k values. This change is intended to normalize the paths so that we
evaluate paths in a way similar to how the config/routes.rb file defines
them.
@codecov
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.22%. Comparing base (c3e73f7) to head (66685db).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5974   +/-   ##
=======================================
  Coverage   97.22%   97.22%           
=======================================
  Files         473      473           
  Lines        9764     9764           
=======================================
  Hits         9493     9493           
  Misses        271      271           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant