Page MenuHomePhabricator

[metricsinfra.karma] alert source links don't work
Closed, ResolvedPublic

Description

Currently, when the alert comes from the metricsinfra prometheus (called 'cloud'), it uses a path prefix '/cloud' to
the alerts, and that does not work.
Removing that path prefix shows the thanos query correctly.

This task is to either remove the '/cloud' from the links shown, or make it work with them.

Example of non-working url:

https://prometheus.wmcloud.org/cloud/graph?g0.expr=puppet_agent_resources_total+%3D%3D+0&g0.tab=1

There's though other prometehus urls that should keep working, for example, if the alert comes from the 'tools'
prometheus instance it will have the '/tools' path:

https://prometheus.wmcloud.org/cloud/graph?g0.expr=sge_queueproblems%7Bproject%3D%22tools%22%2Cstate%3D~%22.%2A%28e%7CE%29.%2A%22%7D&g0.tab=1

Event Timeline

dcaro triaged this task as Medium priority.
dcaro changed the task status from Open to In Progress.Dec 20 2022, 8:52 AM
dcaro moved this task from To refine to Doing on the User-dcaro board.

Change 869187 had a related patch set uploaded (by David Caro; author: David Caro):

[operations/puppet@production] metricsinfra: don't use /cloud path prefix for prometheus

https://gerrit.wikimedia.org/r/869187

Change 869187 merged by David Caro:

[operations/puppet@production] metricsinfra: don't use /cloud path prefix for prometheus

https://gerrit.wikimedia.org/r/869187