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

[alertmanager] add baseURL value #4521

Merged

Conversation

VannTen
Copy link
Contributor

@VannTen VannTen commented May 7, 2024

What this PR does / why we need it

While using extraArgs is possible, baseURL is consistent with the
Prometheus chart and is more discoverable.
The template modification is taken from the prometheus chart.

Checklist

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

While using `extraArgs` is possible, `baseURL` is consistent with the
Prometheus chart and is more discoverable.
The template modification is taken from the prometheus chart.

Signed-off-by: Max Gautier <[email protected]>
@monotek monotek merged commit f29a258 into prometheus-community:main May 8, 2024
4 checks passed
wilfriedroset pushed a commit to wilfriedroset/helm-charts that referenced this pull request May 8, 2024
While using `extraArgs` is possible, `baseURL` is consistent with the
Prometheus chart and is more discoverable.
The template modification is taken from the prometheus chart.

Signed-off-by: Max Gautier <[email protected]>
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.

2 participants