[kube-prometheus-stack] Fix NetworkPolicy for Alertmanager#5359
Merged
jkroepke merged 3 commits intoprometheus-community:mainfrom Mar 3, 2025
Merged
[kube-prometheus-stack] Fix NetworkPolicy for Alertmanager#5359jkroepke merged 3 commits intoprometheus-community:mainfrom
jkroepke merged 3 commits intoprometheus-community:mainfrom
Conversation
a04f109 to
594ae5b
Compare
Set gateway/ingress namespace and podLabel selector to an empty string/object in order to not accidentally expose something unexpected. Move the Loki rule to the additionalIngress examples as Loki is not part of the stack. The 'alertmanager.service.clusterPort' was added as it was not declared in the values and now needs to be enabled explicitly. Issue: prometheus-community#5318 Signed-off-by: Sebastian Gaiser <sebastian.gaiser@hetzner-cloud.de>
594ae5b to
2ec3ce2
Compare
Signed-off-by: Sebastian Gaiser <sebastiangaiser@users.noreply.github.com>
Contributor
Author
|
Ping @jkroepke 😉 |
jkroepke
reviewed
Mar 3, 2025
…icy.yaml Signed-off-by: Sebastian Gaiser <sebastiangaiser@users.noreply.github.com>
Member
|
Thanks |
7 tasks
hedgieinsocks
pushed a commit
to hedgieinsocks/prom-helm-charts
that referenced
this pull request
Apr 27, 2025
…s-community#5359) * [kube-prometheus-stack] Fix NetworkPolicy for Alertmanager Set gateway/ingress namespace and podLabel selector to an empty string/object in order to not accidentally expose something unexpected. Move the Loki rule to the additionalIngress examples as Loki is not part of the stack. The 'alertmanager.service.clusterPort' was added as it was not declared in the values and now needs to be enabled explicitly. Issue: prometheus-community#5318 Signed-off-by: Sebastian Gaiser <sebastian.gaiser@hetzner-cloud.de> * Update charts/kube-prometheus-stack/templates/alertmanager/networkpolicy.yaml Signed-off-by: Sebastian Gaiser <sebastiangaiser@users.noreply.github.com> --------- Signed-off-by: Sebastian Gaiser <sebastian.gaiser@hetzner-cloud.de> Signed-off-by: Sebastian Gaiser <sebastiangaiser@users.noreply.github.com> Signed-off-by: Artyom Babiy <artyom.babiy@gmail.com>
Closed
1 task
This was referenced Jun 5, 2025
Closed
chore(deps): update helm release kube-prometheus-stack to v73 - autoclosed
StefanAbl/init-script#670
Closed
Closed
This was referenced Jun 6, 2025
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
feat(container)!: Update image kube-prometheus-stack to v73 - autoclosed
martinjonsson01/cluster#516
Closed
Closed
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Set gateway/ingress namespace and podLabel selector to an empty string/object in order to not accidentally expose something unexpected. Move the Loki rule to the additionalIngress examples as Loki is not part of the stack. The 'alertmanager.service.clusterPort' was added as it was not declared in the values and now needs to be enabled explicitly.
Issue: #5318
What this PR does / why we need it
Which issue this PR fixes
(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close that issue when PR gets merged)Special notes for your reviewer
Checklist
[prometheus-couchdb-exporter])