Skip to content

[psmdb-db]: pass replset.configuration through tpl function#777

Open
KyriosGN0 wants to merge 2 commits intopercona:mainfrom
KyriosGN0:psmdb-tpl
Open

[psmdb-db]: pass replset.configuration through tpl function#777
KyriosGN0 wants to merge 2 commits intopercona:mainfrom
KyriosGN0:psmdb-tpl

Conversation

@KyriosGN0
Copy link
Copy Markdown
Contributor

fixes #751

@KyriosGN0
Copy link
Copy Markdown
Contributor Author

hey @hors, @nmarukovich @jvpasinatto @eleo007 @valmiranogueira, is it possible to get a review here?
Thanks in advance!

@KyriosGN0
Copy link
Copy Markdown
Contributor Author

hey @hors , @nmarukovich ,@jvpasinatto @eleo007, @valmiranogueira , would it be possible to have a look here? Thanks!

@egegunes
Copy link
Copy Markdown
Contributor

egegunes commented Mar 2, 2026

hi @KyriosGN0, we'll review as soon as possible.

@egegunes
Copy link
Copy Markdown
Contributor

egegunes commented Mar 6, 2026

We are concerned about this change because of fields like auditLog.filter in PSMDB configuration that allows json (for example '{ atype: { $in: ["authenticate"] } }'). We need to ensure this change doesn't affect this kind of configurations. @KyriosGN0 could you please check it?

@KyriosGN0
Copy link
Copy Markdown
Contributor Author

@egegunes i have tested this locally, i was able to template fields in the configuration spec while the auditFilter.json fields remained intact, the filter i used was '{"atype":{"$nin":["clientMetadata","logout","authenticate"]},"param.command":{"$nin":["hello","isMaster","ismaster","ping","replSetHeartbeat"]},"remote.ip":{"$ne":"127.0.0.1"}}'

Signed-off-by: AvivGuiser <avivguiser@gmail.com>
Signed-off-by: AvivGuiser <avivguiser@gmail.com>
name: psmdb-db
home: https://www.percona.com/doc/kubernetes-operator-for-psmongodb/index.html
version: 1.22.1
version: 1.23.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Chart needs patch version bumped like 1.22.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

psmdb-db: support tpl for replicaSet Configuration spec

4 participants