Skip to content

fix: target namespace of grafana secrets#3195

Merged
j-zimnowoda merged 5 commits into
mainfrom
APL-1805
May 6, 2026
Merged

fix: target namespace of grafana secrets#3195
j-zimnowoda merged 5 commits into
mainfrom
APL-1805

Conversation

@merll
Copy link
Copy Markdown
Collaborator

@merll merll commented Apr 30, 2026

📌 Summary

This is a followup of moving platform secrets from values to SealedSecret resources. These corrects the target namespace needed by the platform Grafana instance.

🔍 Reviewer Notes

Grafana currently fails to start in main. With this change, pods should be configured correctly.

🧹 Checklist

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

@svcAPLBot
Copy link
Copy Markdown
Contributor

Comparison of Helm chart templating output:

# raw/templates/resources.yaml

@@ metadata.namespace @@
# external-secrets.io/v1/ExternalSecret/grafana/grafana-loki-datasource-secret
! ± value change
- monitoring
+ grafana

@@ metadata.namespace @@
# external-secrets.io/v1/ExternalSecret/grafana/grafana-oidc-secret
! ± value change
- monitoring
+ grafana

@@ metadata.namespace @@
# external-secrets.io/v1/ExternalSecret/grafana/grafana-admin-secret
! ± value change
- monitoring
+ grafana

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

# values-repo.yaml

@j-zimnowoda j-zimnowoda enabled auto-merge (squash) May 6, 2026 08:21
@j-zimnowoda j-zimnowoda merged commit d7c583a into main May 6, 2026
18 of 19 checks passed
@j-zimnowoda j-zimnowoda deleted the APL-1805 branch May 6, 2026 08:29
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