search

canonical / prometheus-k8s-operator

https://charmhub.io/prometheus-k8s
Apache License 2.0
21 stars 35 forks source link

Fix `stripPrefix` #514

Closed sed-i closed 1 year ago

sed-i commented 1 year ago

Issue

The fix in #512 removed stripPrefix as a workaround, but we still want the stripPrefix functionality.

Solution

Revert #512 and use web.route-prefix.

Drive-by fixes:

Testing Instructions

  1. Relate catalogue - prometheus - traefik and test endpoint with browser. Make sure query results are non empty.
  2. Repeat with TLS.

In tandem with:

sed-i commented 1 year ago

Works unreliably. Converting back to draft.

sed-i commented 1 year ago

Query results were empty because the metrics endpoint path still included the routing prefix :facepalm: