Skip to content

Conversation

@chatton
Copy link
Contributor

@chatton chatton commented Feb 17, 2021

All Submissions:

  • [N/A] Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

This PR upgrades the agent version to a version which supports allowing versions of MongoDB which are not present in the version manifest.

  • The version manifest has been removed from the operator image.
  • A dummy version config is being used in the automation config.
  • The agent version has been upgraded.

closes #255

BUNDLE_IMG ?= controller-bundle:$(VERSION)

# Image URL to use all building/pushing image targets
IMG ?= quay.io/mongodb/mongodb-kubernetes-operator:0.5.0 # replace with localhost:5000/mongodb-kubernetes-operator locally
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removing the official registry as default to prevent any accidental pushes

Copy link
Contributor

Choose a reason for hiding this comment

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

🥳🥳🥳

value: "mongodb-kubernetes-operator"
- name: AGENT_IMAGE # The MongoDB Agent the operator will deploy to manage MongoDB deployments
value: quay.io/mongodb/mongodb-agent:10.19.0.6562-1
value: quay.io/mongodb/mongodb-agent:10.27.0.6772-1
Copy link
Contributor Author

@chatton chatton Feb 17, 2021

Choose a reason for hiding this comment

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

this version of the agent allows us to provide the dummy configuration

Copy link
Contributor

Choose a reason for hiding this comment

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

💯

@chatton chatton marked this pull request as ready for review February 18, 2021 09:30
@chatton chatton requested a review from bznein February 18, 2021 09:30
Copy link
Contributor

@bznein bznein left a comment

Choose a reason for hiding this comment

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

Negative diff, short PR that makes a huge impact on the operator? LGTM!

BUNDLE_IMG ?= controller-bundle:$(VERSION)

# Image URL to use all building/pushing image targets
IMG ?= quay.io/mongodb/mongodb-kubernetes-operator:0.5.0 # replace with localhost:5000/mongodb-kubernetes-operator locally
Copy link
Contributor

Choose a reason for hiding this comment

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

🥳🥳🥳

value: "mongodb-kubernetes-operator"
- name: AGENT_IMAGE # The MongoDB Agent the operator will deploy to manage MongoDB deployments
value: quay.io/mongodb/mongodb-agent:10.19.0.6562-1
value: quay.io/mongodb/mongodb-agent:10.27.0.6772-1
Copy link
Contributor

Choose a reason for hiding this comment

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

💯

@chatton chatton merged commit 6b76b5a into master Feb 18, 2021
@chatton chatton deleted the remove_dependency_on_version_manifest branch February 18, 2021 10:05
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.

clusterConfig failed to validate

3 participants