Skip to content

fix(pmm): remove namespace from ClusterRole metadata#836

Open
franco-viotti wants to merge 3 commits intopercona:mainfrom
franco-viotti:fix/pmm-clusterrole-scope-and-labels
Open

fix(pmm): remove namespace from ClusterRole metadata#836
franco-viotti wants to merge 3 commits intopercona:mainfrom
franco-viotti:fix/pmm-clusterrole-scope-and-labels

Conversation

@franco-viotti
Copy link
Copy Markdown

@franco-viotti franco-viotti commented Apr 15, 2026

Summary

  • remove pmm.includeNamespace from charts/pmm/templates/clusterrole.yaml so rendered ClusterRole does not include metadata.namespace
  • bump charts/pmm/Chart.yaml version from 1.6.0 to 1.6.1

Why

ClusterRole is cluster-scoped and should not carry a namespace. Including metadata.namespace can cause persistent GitOps drift.

Scope

This PR intentionally focuses on the namespace defect and chart version bump only.

Related: #834

@franco-viotti franco-viotti requested a review from a team as a code owner April 15, 2026 09:43
@franco-viotti franco-viotti requested review from JiriCtvrtka and ademidoff and removed request for a team April 15, 2026 09:43
@it-percona-cla
Copy link
Copy Markdown

it-percona-cla commented Apr 15, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ JiriCtvrtka
❌ franco-viotti
You have signed the CLA already but the status is still pending? Let us recheck it.

@JiriCtvrtka
Copy link
Copy Markdown
Contributor

@franco-viotti could you sign CLA license? Thank you.

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.

3 participants