Skip to content

Conversation

jseldess
Copy link
Contributor

This enables them to work on more clusters (including non-GKE clusters)
by default.

This PR replaces #2271.

This enables them to work on more clusters (including non-GKE clusters)
by default.
@jseldess jseldess requested a review from a-robinson December 13, 2017 20:31
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@@ -26,6 +26,31 @@ Choose whether you want to orchestrate CockroachDB with Kubernetes using the hos

The process can take a few minutes, so don't move on to the next step until you see a `Creating cluster cockroachdb...done` message and details about your cluster.

{% if page.secure == true %}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@a-robinson, I added this filter to make sure these RBAC steps don't get pulled into the insecure tutorials.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also made sense to move this content into this include file. But it's only showing up for Hosted GKE. Is RBAC relevant for the other methods as well?

Copy link
Contributor

Choose a reason for hiding this comment

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

RBAC in general is relevant for other methods. These two steps below are a workaround for GKE limitations and aren't relevant for any other environments.

@cockroach-teamcity
Copy link
Member

Copy link
Contributor

@a-robinson a-robinson left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

3 participants