Skip to content

Conversation

@oksana-grishchenko
Copy link
Contributor

@oksana-grishchenko oksana-grishchenko commented Oct 15, 2025

Add PSMDB 1.21.1

Problem:
EVEREST-2173

Related pull requests

CHECKLIST

Helm chart

  • Is the helm chart updated with the new changes? (if applicable)

Jira

  • Is the Jira ticket created and referenced properly?

Tests

  • Is an Integration test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

Comment on lines +679 to +691
// the .Status.BackupConfigHash field appeared in psmdb 1.20.0 so
// we need to check the CR version when we check if the field is populated
versionCheck, err := isCRVersionGreaterOrEqual(psmdbCluster.Spec.CRVersion, "1.20.0")
if err != nil {
return false, err
}
// Requeue if the pbm is not configured yet in psmdb 1.20.0+.
// An indicator that the pbm is configured in 1.20.0+ is the non-empty psmdb.Status.BackupConfigHash
// NOTE: in the future, to add support for multiple storages we would need not only to check the BackupConfigHash but also understand if
// it has changed to be able to create on-demand backups to a new storage.
if psmdbCluster.Status.BackupConfigHash == "" && versionCheck {
return true, nil
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

still relevant although we skipped 1.20 and integrate 1.21 now

@oksana-grishchenko oksana-grishchenko added the test/e2e Run full E2E test suite label Oct 22, 2025
@oksana-grishchenko oksana-grishchenko changed the title EVEREST-2173 Add PSMDB 1.21.0 EVEREST-2173 Add PSMDB 1.21.1 Oct 31, 2025
@oksana-grishchenko oksana-grishchenko marked this pull request as ready for review November 4, 2025 05:41
@oksana-grishchenko oksana-grishchenko requested a review from a team as a code owner November 4, 2025 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test/e2e Run full E2E test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants