envoyproxy / envoyproxy/envoy

[stats, xds] disable per-resource config update stats tracking, add per-API update_duration tracking (if necessary).

Open
#23,979 10 comments 0 reactions 0 assignees View on GitHub
area/perf area/stats enhancement no stalebot
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 22h
Merged PRs (30d)
430

Description

*Title*: disable per-resource update_duration tracking, add per-API update_duration tracking (if necessary).

*Description*: right now there is a histogram tracking per-resource update duration, i.e. how long it takes Envoy to consume and update a resource's config.

This stat is of less value when there are tons of resources in Envoy, what matters more is how long does Envoy take to load each discoveryResponse.

OTOH, histograms are known to consume lots of RAM and CPU.

We should allow users to disable per-resource update-duration tracking (with a runtime flag) and maybe add a per-API level update-duration tracking instead.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.