Skip to content

Commit

Permalink
dex: 🤌 add 1s, 5s buckets to DEX_PATH_SEARCH_RELAX_PATH_DURATION
Browse files Browse the repository at this point in the history
this is a known source of latency. let's use some extra wide buckets to
capture any instances of additional >250ms latency.
  • Loading branch information
cratelyn committed Jun 6, 2024
1 parent 1e71cda commit d159da2
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion crates/core/component/dex/src/component/metrics.rs
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,14 @@ impl PrometheusBuilderExt for metrics_exporter_prometheus::PrometheusBuilder {
)?
.set_buckets_for_metric(
Full(DEX_PATH_SEARCH_RELAX_PATH_DURATION.to_owned()),
GENERIC_DEX_BUCKETS,
{
// Add some larger buckets to the histogram tracking how long it takes to relax a
// path. See https://github.com/penumbra-zone/penumbra/issues/4464.
let mut buckets = GENERIC_DEX_BUCKETS.to_vec();
buckets.extend_from_slice(&[1.000, 5.000]);
buckets
}
.as_slice(),
)?
.set_buckets_for_metric(
Full(DEX_ROUTE_FILL_DURATION.to_owned()),
Expand Down

0 comments on commit d159da2

Please sign in to comment.