Skip to content
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

DOC-441 Added a quick start guide for generic K8s #1174

Merged
merged 5 commits into from
Feb 1, 2021

Conversation

bmansheim
Copy link
Contributor

No description provided.

@bmansheim bmansheim added enhancement Additional information or functionality p_Platforms labels Jan 6, 2021
@alexmilowski alexmilowski requested a review from banker January 12, 2021 23:41
The Redis Enterprise Operator manages a single Redis Enterprise cluster in a single namespace.

While the operator can watch additional namespaces for consumer namespaces, the Redis Enterprise Operator
and cluster must exist in the same namespace.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't understand this sentence. Can you confirm that it's correctly worded and necessary? @alexmilowski @amiramm

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's remove it. There isn't enough context in this guide about consumer namespaces and we need to keep it simple as this should be a simple guide followed almost to the letter to get started.

@banker
Copy link
Contributor

banker commented Jan 25, 2021

@amiramm or @alexmilowski. I've finished reviewing this document. Please see my one outstanding question, and please read the doc once more, post my edits. I'm happy to merge.

@banker banker merged commit bc795c3 into master Feb 1, 2021
@banker banker deleted the DOC-441-quick-start-guide-k8s branch February 1, 2021 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Additional information or functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants