Skip to content

CFP-26521: Expose URL path in Hubble HTTP metrics#100

Open
amaanx86 wants to merge 1 commit into
cilium:mainfrom
amaanx86:feature/cfp-26521-http-path-metrics
Open

CFP-26521: Expose URL path in Hubble HTTP metrics#100
amaanx86 wants to merge 1 commit into
cilium:mainfrom
amaanx86:feature/cfp-26521-http-path-metrics

Conversation

@amaanx86

Copy link
Copy Markdown

tl;dr: add an opt-in path label to the Hubble HTTP Prometheus metrics, with operator-supplied templates that collapse variable URL segments (IDs, slugs) into stable placeholders, and a hard cardinality cap.

The original issue was closed in 2023 on cardinality grounds and reopened in 2024 once the discussion moved toward a templated, opt-in design. This CFP writes that design down so implementation work can start. Key points:

  • New optional pathContext block in the HTTP metric config; fits both the legacy hubble-metrics config string and the dynamic config map from CFP-30788.
  • Templates are operator-supplied, matched in order, with a maxCardinality ceiling and a configurable miss policy (drop_label, drop_sample, bucket_other).
  • Two small self-observability metrics for the matcher itself.
  • Backwards compatible: if pathContext is unset, the metric labels are identical to today.

Pulling templates from L7 CiliumNetworkPolicy rules (suggested by @apt-itude on the issue) is discussed as an alternative configuration source and deferred to a follow-up CFP.

Cilium issue: cilium/cilium#26521

Signed-off-by: Amaan Ul Haq Siddiqui <amaanulhaq.s@outlook.com>
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