Skip to content

Conversation

@nammn
Copy link
Collaborator

@nammn nammn commented Apr 29, 2025

Summary

This pull request primarily focuses on renaming resources and references from "mongodb-enterprise" to "mongodb-kubernetes" across the codebase. The changes ensure consistency in naming conventions and align with the updated project branding. Below are the key changes grouped by theme:

Resource and Binary Renaming

  • Updated the output binary name in Makefile from mongodb-enterprise-operator to mongodb-kubernetes-operator.
  • Changed the command in manager.yaml to use /usr/local/bin/mongodb-kubernetes-operator instead of /usr/local/bin/mongodb-enterprise-operator.
  • Updated the Helm chart template to reflect the new binary name mongodb-kubernetes-operator.

Repository and Documentation Updates

  • Updated repository links in various files (e.g., release.md, contributing.md, README.md) to point to https://github.com/mongodb/mongodb-kubernetes instead of https://github.com/mongodb/mongodb-enterprise-kubernetes. [1] [2] [3]
  • Adjusted references in README.md for the database Docker image to use mongodb-kubernetes-database in place of mongodb-enterprise-database. [1] [2] [3]

Dockerfile Adjustments

  • Updated Dockerfiles to copy licenses and binaries for mongodb-kubernetes instead of mongodb-enterprise. [1] [2] [3]
  • Renamed directories and paths in Docker-related files to align with the mongodb-kubernetes naming convention. [1] [2]

Inventory and Image Naming

  • Updated image registry paths in inventory files (e.g., database.yaml, test.yaml) to use mongodb-kubernetes instead of mongodb-enterprise. [1] [2] [3] [4]

Miscellaneous

  • Modified the generate_ssdlc_report.py script to use mongodb-kubernetes for image names, except for "ops-manager" which retains its original naming convention.
  • Removed the unused e2e-inventory.yaml file from the mongodb-community-operator directory.

These changes collectively ensure consistency in naming and branding across the project while removing outdated references to "mongodb-enterprise."

Proof of Work

  • green ci

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 changed the base branch from mck-convert-enterprise to master April 29, 2025 12:47
@nammn
Copy link
Collaborator Author

nammn commented Apr 29, 2025

evergreen retry

@nammn nammn marked this pull request as ready for review April 30, 2025 15:17
@nammn nammn requested a review from a team as a code owner April 30, 2025 15:17
@nammn nammn requested review from Julien-Ben and lsierant April 30, 2025 15:17
@nammn nammn changed the title rename rest to mongodb-kubernetes as well standardize references from "mongodb-enterprise" to "mongodb-kubernetes Apr 30, 2025
Copy link
Member

@mircea-cosbuc mircea-cosbuc left a comment

Choose a reason for hiding this comment

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

LGTM

@mircea-cosbuc mircea-cosbuc requested a review from lucian-tosa May 1, 2025 13:10
@mircea-cosbuc
Copy link
Member

@lucian-tosa Comments were addressed in 4291950 , please take another look.

@mircea-cosbuc mircea-cosbuc merged commit ccf22c8 into master May 1, 2025
33 checks passed
@mircea-cosbuc mircea-cosbuc deleted the mck-reg-refs-2 branch May 1, 2025 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants