Skip to content

Conversation

@lsierant
Copy link
Contributor

Summary

This is a no-op refactor.

  • Merged mongodb-community-operator/pkg/kube/statefulset into pkg/statefulset. All other mongodb-community-operator/pkg/kube packages should be moved the same way, but this PR doesn't touch them.
    • some merged unit tests had the same names; conflicting names from MCO were renamed with _MCO suffix.
  • extracted general resource and status functions from controllers/operator into pkg/kube/commoncontroller so that the functions can be shared by other controllers that are in different packages

Proof of Work

Green tests.

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 checked for release_note changes?

Reminder (Please remove this when merging)

  • Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
  • Our Short Guide for PRs: Link
  • Remember the following Communication Standards - use comment prefixes for clarity:
    • blocking: Must be addressed before approval.
    • follow-up: Can be addressed in a later PR or ticket.
    • q: Clarifying question.
    • nit: Non-blocking suggestions.
    • note: Side-note, non-actionable. Example: Praise
    • --> no prefix is considered a question
@nammn nammn self-requested a review April 23, 2025 12:42
@lsierant lsierant merged commit b7fc44d into master Apr 24, 2025
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants