Skip to content

CLOUDP-324655 - fix MongoDBMultiCluster not ready if clusterSpecList contains 0 members #484

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 2, 2025
Merged

CLOUDP-324655 - fix MongoDBMultiCluster not ready if clusterSpecList contains 0 members #484

merged 3 commits into from
Oct 2, 2025

Conversation

MaciejKaras
Copy link
Collaborator

@MaciejKaras MaciejKaras commented Oct 1, 2025 •
edited
Loading

Summary

Fixes issue in MongoDBMultiCluster not ready if clusterSpecList contains 0 members.

Proof of Work

Passing CI with new test case where we scale down to 0 members in one cluster.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you added changelog file?

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 1, 2025 •
edited
Loading

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.5.0 Release Notes

New Features

  • Improve automation agent certificate rotation: the agent now restarts automatically when its certificate is renewed, ensuring smooth operation without manual intervention and allowing seamless certificate updates without requiring manual Pod restarts.

Bug Fixes

  • MongoDBMultiCluster: fix resource stuck in Pending state if any clusterSpecList item has 0 members. After the fix, a value of 0 members is handled correctly, similarly to how it's done in the MongoDB resource.

@MaciejKaras MaciejKaras changed the title CLOUDP-324655 - fix MongoDBMultiCluster not ready if clusterSpec contains 0 members CLOUDP-324655 - fix MongoDBMultiCluster not ready if clusterSpecList contains 0 members Oct 1, 2025
@MaciejKaras MaciejKaras marked this pull request as ready for review October 1, 2025 12:11
@MaciejKaras MaciejKaras requested review from vinilage and a team as code owners October 1, 2025 12:11
Julien-Ben
Julien-Ben approved these changes Oct 1, 2025
View reviewed changes
Copy link
Collaborator

@Julien-Ben Julien-Ben left a comment

Choose a reason for hiding this comment

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

Great fix !

vinilage
vinilage approved these changes Oct 2, 2025
View reviewed changes
Copy link
Collaborator

@vinilage vinilage left a comment

Choose a reason for hiding this comment

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

LGTM!

@MaciejKaras MaciejKaras merged commit b3ea899 into master Oct 2, 2025
37 checks passed
@MaciejKaras MaciejKaras deleted the maciejk/CLOUDP-324655 branch October 2, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anandsyncs anandsyncs anandsyncs approved these changes

@Julien-Ben Julien-Ben Julien-Ben approved these changes

@vinilage vinilage vinilage approved these changes

@lucian-tosa lucian-tosa Awaiting requested review from lucian-tosa lucian-tosa is a code owner automatically assigned from mongodb/kubernetes-hosted

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MaciejKaras @anandsyncs @Julien-Ben @vinilage