Commit e90987e
fix(mongodb-store): bump psmdb-operator to 1.22.0
Percona's server-mongodb-operator 1.21.x has a bug in its cert-manager
TLS handling: when cert-manager is run with
`--enable-certificate-owner-ref=true`, it sets the Certificate object
as the controller-owner of the Secrets it issues. The operator's
WaitForCerts() in that version unconditionally tries to also set the
PerconaServerMongoDB CR as controller-owner of the same Secret,
which Kubernetes rejects since an object can only have one
controller-owner reference. This makes cert-manager-issued TLS
never come up when that cert-manager flag is enabled, on every
reconcile, with no automatic recovery:
TLS secrets handler: "create ssl by cert-manager: update cert
mangager certs: failed to apply cert-manager certificates: failed
to wait for ca cert: set controller reference: Object
<ns>/<cluster>-ca-cert is already owned by another Certificate
controller <cluster>-ca-cert". Please create your TLS secret
<cluster>-ssl manually or setup cert-manager correctly
This is a known upstream issue, fixed in 1.22.0 by adding an
IsControlledBy() check before attempting to set the owner reference:
percona/percona-server-mongodb-operator#1981
Bumps the vendored psmdb-operator subchart's image tag from 1.21.0 to
1.22.0 and regenerates Chart.lock. The vendored CRDs are unchanged;
1.22.0's release notes list only additive, optional features
(service-mesh appProtocol, restore replset remapping, disable-auth
option, custom env vars), none of which mongodb-store's psmdb-db
values currently use, so running 1.22.0 against the existing CRD
schema is expected to be safe.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>1 parent c9c73b5 commit e90987e
4 files changed
Lines changed: 7 additions & 7 deletions
File tree
- distros/kubernetes/nvsentinel/charts/mongodb-store
- charts/psmdb-operator
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | | - | |
12 | | - | |
| 11 | + | |
| 12 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
| 2 | + | |
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | | - | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
0 commit comments