-
Notifications
You must be signed in to change notification settings - Fork 521
(DOCSP-13959): cross-namespace deployment docs #319
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
jeff-allen-mongo
left a comment
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.
Some of the links don't seem to work. Content looks solid though.
jeff-allen-mongo
left a comment
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 with one non-blocking comment.
| - mongodb-kubernetes-operator | ||
| imagePullPolicy: Always | ||
| env: | ||
| - name: WATCH_NAMESPACE |
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.
Can you highlight lines in markup? Might be helpful here if so, but definitely not necessary.
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.
I don't think there's a simple way to do this. I did a little research but came up empty.
docs/install-upgrade.md
Outdated
|
|
||
| ### Understand Deployment Scopes | ||
|
|
||
| You can deploy the MongoDB Community Kubernetes Operator with different scopes based on where you want to deploy Ops Manager and MongoDB Kubernetes resources: |
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.
The Community Operator doesn't support Ops Manager :)
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.
+1
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.
oof. copy / paste mistake from the enterprise operator. fixed.
alyacb
left a comment
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.
Looks good, but the reference to Ops Manager should be removed before merging.
alyacb
left a comment
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! 🎉
1ede7a6 to
62d31bd
Compare
All Submissions:
closes #XXXXin your comment to auto-close the issue that your PR fixes (if such).closes #316
JIRA