-
Notifications
You must be signed in to change notification settings - Fork 81
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
CLOUDP-222191: Update quick start guide and provided samples #1421
Conversation
What do we do about the quickstart guide in the CSV? https://github.com/mongodb/mongodb-atlas-kubernetes/blob/main/config/manifests/bases/mongodb-atlas-kubernetes.clusterserviceversion.yaml#L70-L176 |
@helderjs that's a great start, thank you! I have a couple of comments and a question regarding the CSV. |
Thanks for reminding me about it. Looks like it was never updated, the information there was from before GA 😬 |
README.md
Outdated
Before you install the MongoDB Atlas Operator using `kubectl`, you must: | ||
|
||
1. Install [kubectl](https://kubernetes.io/docs/tasks/tools/install-kubectl/). | ||
2. Have Kubernetes installed, i.e. [Kind](https://kind.sigs.k8s.io/) |
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.
Or have access to a kubernetes cluster configuration right?
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, I assume we have followed the new steps successfully right?
config/manifests/bases/mongodb-atlas-kubernetes.clusterserviceversion.yaml
Outdated
Show resolved
Hide resolved
The CSV contains copy&paste from the README. We need a ratified version by @dan-mckean here instead. |
e7d6248
to
9aac047
Compare
Co-authored-by: Sergiusz Urbaniak <[email protected]>
All Submissions:
closes #XXXX
in your comment to auto-close the issue that your PR fixes (if there is one).I replaced the hard-coded examples with references from inside our config sample folder. Ensuring both work