Skip to content

fix: failed connection between grafana and alertmanager#3222

Merged
j-zimnowoda merged 2 commits into
mainfrom
APL-1817
May 8, 2026
Merged

fix: failed connection between grafana and alertmanager#3222
j-zimnowoda merged 2 commits into
mainfrom
APL-1817

Conversation

@ferruhcihan
Copy link
Copy Markdown
Contributor

📌 Summary

This PR fixes Grafana's Alertmanager datasource access by adding grafana/po-grafana to the monitoring AuthorizationPolicy. (Please check #3068)

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

@svcAPLBot
Copy link
Copy Markdown
Contributor

Comparison of Helm chart templating output:

# kube-prometheus-stack/templates/extra-objects.yaml

@@ spec.rules.0.from @@
# security.istio.io/v1/AuthorizationPolicy/monitoring-auth-require-jwt
! - one list entry removed:
- - source:
-     serviceAccounts:
-     - monitoring/po-prometheus
! + one list entry added:
+ - source:
+     serviceAccounts:
+     - grafana/po-grafana
+     - monitoring/po-prometheus

# rabbitmq-cluster-operator/templates/messaging-topology-operator/validating-webhook-configuration.yaml

# values-repo.yaml

@merll
Copy link
Copy Markdown
Collaborator

merll commented May 8, 2026

Is this working for the team Alertmanager? Or do we need to add the exemption there as well?

@merll
Copy link
Copy Markdown
Collaborator

merll commented May 8, 2026

Is this working for the team Alertmanager? Or do we need to add the exemption there as well?

nvm, that should not be an issue. The Alertmanager of each team is only configured in the same team's Grafana instance. They all run in the same namespace, which already has the rules set accordingly.

@j-zimnowoda j-zimnowoda enabled auto-merge (squash) May 8, 2026 09:12
@j-zimnowoda j-zimnowoda merged commit eb0cf03 into main May 8, 2026
14 checks passed
@j-zimnowoda j-zimnowoda deleted the APL-1817 branch May 8, 2026 09:15
ferruhcihan added a commit that referenced this pull request May 28, 2026
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.

4 participants