Skip to content

Conversation

@anandsyncs
Copy link
Contributor

@anandsyncs anandsyncs commented Oct 26, 2025

Summary

This pull request introduces support for tracking and updating the reconciled MongoDB Search version within the operator and its CRD status. The changes ensure that the Status.Version field is set only after the StatefulSet becomes ready, and add mechanisms to pass and validate the version throughout the reconciliation workflow. Additional test coverage verifies the correct behavior before and after readiness.

CRD and Status Enhancements:

  • Added a new Version column to the MongoDBSearch CRD for visibility in kubectl get output.
  • Updated the UpdateStatus method to accept and set the MongoDBSearchVersionOption, allowing the controller to update the reconciled version in status.
  • Introduced MongoDBSearchVersionOption and related helpers in status_options.go to encapsulate version updates for status.

Controller and Reconciliation Logic:

  • Refactored reconciliation to extract and validate the mongot version, and propagate it to status only after the StatefulSet is marked ready. [1] [2]
  • Improved image tag extraction logic to reliably determine the version from container image references. [1] [2]

Testing Improvements:

  • Enhanced the reconciliation test to verify that Status.Version remains empty before readiness and is set correctly after the StatefulSet is ready; added a helper to simulate readiness. [1] [2] [3] [4]

Proof of Work

Tests pass

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
Copy link

github-actions bot commented Oct 26, 2025

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

MCK 1.6.0 Release Notes

New Features

  • MongoDBCommunity: Added support to configure custom cluster domain via newly introduced spec.clusterDomain resource field. If spec.clusterDomain is not set, environment variable CLUSTER_DOMAIN is used as cluster domain. If the environment variable CLUSTER_DOMAIN is also not set, operator falls back to cluster.local as default cluster domain.
  • Helm Chart: Introduced two new helm fields operator.podSecurityContext and operator.securityContext that can be used to configure securityContext for Operator deployment through Helm Chart.

Bug Fixes

  • Fixed parsing of the customEnvVars Helm value when values contain = characters.
  • ReplicaSet: Blocked disabling TLS and changing member count simultaneously. These operations must now be applied separately to prevent configuration inconsistencies.
  • MongoDBSearch now records the reconciled mongot version in status and exposes it via a dedicated kubectl print column.

@anandsyncs anandsyncs force-pushed the CLOUDP-327089-add-status-field branch from 2536abe to ae52bb4 Compare October 28, 2025 09:57
@anandsyncs anandsyncs marked this pull request as ready for review October 28, 2025 09:58
@anandsyncs anandsyncs requested review from a team and vinilage as code owners October 28, 2025 09:58
@anandsyncs anandsyncs requested review from m1kola and nammn October 28, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants