-
Notifications
You must be signed in to change notification settings - Fork 8
Rename operator image #25
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.
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR renames the operator image to use the new repo naming convention ("mongodb-kubernetes") across various configuration files and scripts.
- Updated image references in Evergreen cleanup script, Kubernetes manifests, pipeline configuration, Helm chart values, and test setup.
- Ensured consistency across YAML files, build configurations, and environment variable-driven test configurations.
Reviewed Changes
Copilot reviewed 10 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
scripts/evergreen/periodic-cleanup-aws.py | Updated operator image reference |
public/mongodb-enterprise.yaml | Changed image string to new operator image |
public/mongodb-enterprise-openshift.yaml | Changed image string to new operator image |
public/mongodb-enterprise-multi-cluster.yaml | Changed image string to new operator image |
pipeline.py | Updated image_config call for operator image |
mongodb-community-operator/test/e2e/setup/test_config.go | Updated test configuration to use new operator image |
inventory.yaml | Updated image registry references |
helm_chart/values.yaml | Renamed operator image in chart values; deployment name remains unchanged |
config/manifests/bases/mongodb-kubernetes.clusterserviceversion.yaml | Updated containerImage to new operator image |
config/manager/manager.yaml | Updated manager image to new operator image |
Files not reviewed (4)
- mongodb-community-operator/scripts/ci/config.json: Language not supported
- public/architectures/setup-multi-cluster/setup-operator/output/0210_helm_install_operator.out: Language not supported
- scripts/evergreen/operator-sdk/prepare-openshift-bundles-for-e2e.sh: Language not supported
- scripts/evergreen/operator-sdk/prepare-openshift-bundles.sh: Language not supported
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
2d98833
to
2222407
Compare
I rebased on top of master because there was a conflict in |
# Summary * Fixes operator image keys in build scripts. This was missed in #25 (see [this relevant change](https://github.com/mongodb/mongodb-kubernetes/pull/25/files#diff-6b92e3e77956f8020482ca91889a24f3411c9a4854f2aaaac30ee504ef26ede4R650)). * Fixes remaining references to MongoDB Enterprise Kubernetes Operator Quay images that block the release ## Proof of Work * CI for this PR must be green * Release `release_images` variant must be green (see [this run](https://spruce.mongodb.com/version/680943253fab280007839c8e/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC)) ``` evergreen patch --path .evergreen.yml -p mongodb-kubernetes -d "Test MCK release: #33" -v release_images -t all -f -y -u --browse ```
Summary
Use new repo for the operator image
Proof of Work
CI must be green (except for OLM upgrade jobs which will be fixed in #22)