percona / percona-xtradb-cluster-operator

Percona Operator for MySQL based on Percona XtraDB Cluster
https://www.percona.com/doc/kubernetes-operator-for-pxc/index.html
Apache License 2.0
511 stars 187 forks source link

pxc-db pmm configure pmmserverkey #1586

Open weiss-m opened 8 months ago

weiss-m commented 8 months ago

We use the pxc operator password auto generation feature for pxc accounts like root, xtrabackup ...

Unfortunately, pxc operator can't auto generate pmmserverkey. So we have to add the pmmserverkey manually to the existing k8s secret which already contains the auto generated passwords for root,xtrabackup ... https://github.com/percona/percona-xtradb-cluster-operator/blob/main/deploy/secrets.yaml#L11 https://docs.percona.com/percona-operator-for-mysql/pxc/monitoring.html#install-pmm-server

This is a manual step which we would like to avoid. We are using argocd for deployment of several pxc clusters. So, one option would be, to use kustomize to add pmmserverkey to the existing k8s secret. This is not an option for us because we don't store passwords in clear text in git. We would like to use sealed secrets, but sealed secrets doesn't support addition of a key, value pair to an existing k8s secret.

Would it be possible to store auto generated and manual created credentials in separate k8s secrets?

spron-in commented 8 months ago

Hey @weiss-m ,

so let me see if I understand the problem:

  1. You deploy PXC with argocd
  2. You rely on auto-generated secrets for MySQL
  3. At the same time you use PMM and you don't have an easy automated way to add pmmserverkey into the generated secret.

Let me think how we can do that. @hors @egegunes any thoughts?

weiss-m commented 8 months ago

Just for my understanding, is the following assumption correct? The operator uses cluster-secrets as input and applies the password accordingly to cluster-secrets in the database and afterward applying it writes the passwords in secret internal-cluster ? So in cluster-secrets is always the desired state? And in internal-cluster the current state?

spron-in commented 8 months ago

Yes, you are correct.