K8SPSMDB-1598 Fix replset.arbiter.affinity check#802
Merged
jvpasinatto merged 2 commits intorelease-psmdb-1.22.0from Feb 23, 2026
Merged
K8SPSMDB-1598 Fix replset.arbiter.affinity check#802jvpasinatto merged 2 commits intorelease-psmdb-1.22.0from
jvpasinatto merged 2 commits intorelease-psmdb-1.22.0from
Conversation
jvpasinatto
requested review from
eleo007,
hors,
nmarukovich and
valmiranogueira
as code owners
eleo007
approved these changes
Feb 23, 2026
valmiranogueira
approved these changes
Feb 23, 2026
jvpasinatto
added a commit
that referenced
this pull request
Feb 25, 2026
* Update PSMDB chart versions for 1.22.0 release * add revisionhistorylimit Signed-off-by: drfaust92 <ilia.lazebnik@gmail.com> * CR comments Signed-off-by: drfaust92 <ilia.lazebnik@gmail.com> * K8SPSMDB-1470: Add CRD sub-chart to enable CRD upgrades (#753) * feat: Add CRD sub-chart for psmdb-operator to enable CRD upgrades - Create psmdb-operator-crds sub-chart with individual CRD templates - Add CRD sub-chart as optional dependency in psmdb-operator Chart.yaml - Add crds.enabled configuration option (default: false for backward compatibility) - Update README with CRD upgrade instructions - Move crds section to end of values.yaml for better organization - Follows Everest pattern for consistency across Percona charts This allows users to: - Upgrade CRDs via helm upgrade (previously not possible) - Install CRDs separately for better lifecycle management - Use with GitOps tools (ArgoCD, FluxCD) that support server-side apply Resolves: K8SPSMDB-1470 * docs: Add documentation for CRD ownership limitation - Add 'Important Limitation' section explaining CRD ownership metadata issue - Add troubleshooting section with exact error messages and solutions - Add warning in operator README about enabling crds.enabled=true with existing CRDs - Provide clear guidance on when and how to take ownership of existing CRDs This helps users understand why enabling crds.enabled=true fails when CRDs were previously installed via the crds/ directory and provides solutions. * chore: Update chart versions to 1.22.0 for release-psmdb-1.22.0 compatibility - Update psmdb-db chart to version 1.22.0 - Update psmdb-operator chart to version 1.22.0 - Update psmdb-operator-crds chart to version 1.22.0 - Update all version references in README files - Update default image tags in values.yaml files * fix: Remove extra blank line in psmdb-operator-crds Chart.yaml * fix: Remove extra blank lines in psmdb-operator-crds values.yaml * fix: Update Chart.lock for psmdb-operator with new CRD chart version * chore: Update app.kubernetes.io/version to v1.22.0 in CRD templates * chore: Add bogdanjeler-ev to psmdb-operator-crds maintainers * fix: Resolve merge conflict in psmdb-operator README.md - Merge revisionHistoryLimit parameter from target branch - Keep maxConcurrentReconciles parameter from feature branch - Remove duplicate podAnnotations entry - Use consistent formatting from target branch * docs: Add bogdanjeler-ev as maintainer to psmdb-operator-crds README * docs: Remove trailing empty lines from psmdb-operator-crds README * fix: Address PR feedback - sync CRDs, remove 'recommended' language, add validation - Sync charts/psmdb-operator/crds/crd.yaml with CRD templates - Remove 'Recommended for Production' language from READMEs (first release) - Add GitHub Action to validate CRD sync on PRs - Add note about default behavior discussion in values.yaml - Ensure CRD files stay in sync for future releases * fix: Remove push trigger from CRD sync check workflow Only run validation on pull requests, not on pushes to main branch. * Add version sync validation to CRD sync check workflow - Validate that psmdb-operator-crds chart version matches operator appVersion - Validate that CRD chart appVersion matches operator appVersion - Validate that dependency version matches CRD chart version - Add clear notes that this rule applies ONLY to psmdb-operator-crds - Other charts (e.g., psmdb-db) can have independent chart versions * Fix CRD sync check: match original format without separators The original crd.yaml file concatenates CRDs directly without '---' separators or newlines between them (format: 'status: {}apiVersion: ...'). - Remove '---' separators when combining templates - Remove trailing newline from each CRD file before concatenating - Handle Windows line endings properly - Update sync instructions to match the correct format * Remove Maintainers section from psmdb-operator-crds README * Fix CRD sync check: remove trailing newline to match original format The original crd.yaml file has no trailing newline at the end. Remove trailing newlines from both files before comparison to ensure they match exactly. * update crds * update maintainers * Add condiotionally revisionHistoryLimit * Fix lint and update CRDs * Fix crd chart test * Fix and simplify CRD sync * K8SPSMDB-1472 pvc auto resize (#763) * K8SPSMDB-1472 pvc auto resize * update the latest cr options * update crds * update crd with new line * Add missing space in comment --------- Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * add minio native example in values.yaml (#765) Signed-off-by: Mayank Shah <mayank.shah@percona.com> Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * K8SPSMDB-1448: add `vault` section (#759) * K8SPSMDB-1448: add `vault` section https://perconadev.atlassian.net/browse/K8SPSMDB-1448 * fix vault template --------- Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * K8SPSMDB-1531: add `hookScript` sections (#758) * K8SPSMDB-1531: add `hookScript` sections https://perconadev.atlassian.net/browse/K8SPSMDB-1531 * remove vim comment --------- Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * K8SPSMDB-1539: add `namespaceOverride` to psmdb operator (#743) https://perconadev.atlassian.net/browse/K8SPSMDB-1539 Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * K8SPSMDB-1418 add ca bundles (#778) * K8SPSMDB-1418 add ca bundles * fix comment spacing * fix more comment spacing --------- Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * update charts and notes * update image versions, cw-rbac and fixes * Add *-operator-crds to exclude list for tests (#801) * K8SPSMDB-1598 Fix replset.arbiter.affinity check (#802) Co-authored-by: Valmira Nogueira Corrêa <valmiranogueira@hotmail.com> * fix: hidden nodes from a replicaset missing serviceAccount option (#768) Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> Co-authored-by: Valmira Nogueira Corrêa <valmiranogueira@hotmail.com> * K8SPSMDB-1470: revert dependency between crds and operator charts (#800) * K8SPSMDB-1470: revert dependency between crds and operator charts * Fix description * Extend README --------- Co-authored-by: Valmira Nogueira Corrêa <valmiranogueira@hotmail.com> Co-authored-by: Julio Pasinatto <julio.pasinatto@percona.com> * replace null fields with empty list/dict --------- Signed-off-by: drfaust92 <ilia.lazebnik@gmail.com> Signed-off-by: Mayank Shah <mayank.shah@percona.com> Co-authored-by: drfaust92 <ilia.lazebnik@gmail.com> Co-authored-by: Valmira Nogueira Corrêa <valmiranogueira@hotmail.com> Co-authored-by: bogdanjeler-ev <bogdan.jeler@percona.com> Co-authored-by: George Kechagias <geo.kechagias@gmail.com> Co-authored-by: Mayank Shah <mayankshah1614@gmail.com> Co-authored-by: Andrii Dema <a.dema@jazzserve.com> Co-authored-by: Natalia Marukovich <nmarukovich@gmail.com> Co-authored-by: Eleonora Zinchenko <eleonora.zinchenko@percona.com> Co-authored-by: Gabi Hulea <gabi.hulea@door.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.