Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong Alert's source URL #206

Closed
Abuelodelanada opened this issue Nov 24, 2023 · 5 comments · Fixed by canonical/prometheus-k8s-operator#635
Closed

Wrong Alert's source URL #206

Abuelodelanada opened this issue Nov 24, 2023 · 5 comments · Fixed by canonical/prometheus-k8s-operator#635

Comments

@Abuelodelanada
Copy link
Contributor

Bug Description

Despite of the fact Alertmanager and Prometheus are behind an ingress, the source of the alert use k8s-based URL

image

To Reproduce

  1. Deploy cos-lite bundle
  2. In Alertmanager UI check for the Source of the alert.

Environment

  • juju 3.1.6
  • cos-lite bundle

Relevant log output

-

Additional context

No response

@mcarvalhor
Copy link
Member

Affects us too.

@cbartz
Copy link

cbartz commented Apr 18, 2024

This affects us too, and also the ExternalURL (used in backlinks for the alert title in slack notifications) is set to the k8s-based URL. I wonder if changing https://github.com/canonical/alertmanager-k8s-operator/blob/main/src/charm.py#L168 to use _external_url could at least fix the ExternalURL issue.

@mcarvalhor
Copy link
Member

Also, this may be obvious but out of interest: this wrong URL escalates to PagerDuty/monitoring services.
So on call engineers need to manually copy the link and fix the URL before pasting to the browser.

What I mean is: indeed an annoying bug, and during outages it can contribute to resolution delays.

@marcusboden
Copy link

We're also affected by this. Having to fix the URLs each time is time consuming in emergencies.

@lucabello
Copy link
Contributor

Increasing the priority on this one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants