Skip to content

Conversation

seanmwinn
Copy link
Contributor

Signed-off-by: Sean Winn sean@isovalent.com

Adds a getting started guide for Rancher Kubernetes Engine

Fixes: #11380

Adds a Getting Started Guide for Rancher Kubernetes Engine

@seanmwinn seanmwinn requested a review from a team as a code owner March 11, 2021 22:01
@seanmwinn seanmwinn requested a review from qmonnet March 11, 2021 22:01
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 11, 2021
@seanmwinn seanmwinn added the release-note/misc This PR makes changes that have no direct user impact. label Mar 11, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 11, 2021
@seanmwinn seanmwinn changed the title adds a rancher kubernetes engine installation guide docs: adds a rancher kubernetes engine installation guide Mar 11, 2021
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Looks good!
Some suggestions below, but they're not blocking.

Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Looks like the feedback was all pretty straightforward and should be able to be addressed without too much effort. Would you mind fixing it up before we merge? Otherwise we tend to end up just ignoring any feedback and never following up.

(Maybe you intended to push an updated version but pushed the old version instead? I see the commit update in the page above but the changes don't appear to be present in the PR.)

Signed-off-by: Sean Winn <sean@isovalent.com>
@seanmwinn seanmwinn force-pushed the pr/seanw/install_rke branch from a4a48fb to ae68c89 Compare March 12, 2021 23:07
@seanmwinn
Copy link
Contributor Author

@joestringer I simply missed adding the file, re-pushed

@seanmwinn seanmwinn requested a review from joestringer March 12, 2021 23:08
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Ah, that explains the mixup. LGTM, thanks!

@qmonnet qmonnet added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 14, 2021
@kkourt kkourt merged commit ab4bafc into master Mar 15, 2021
@kkourt kkourt deleted the pr/seanw/install_rke branch March 15, 2021 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs: Add Integration Guide for Rancher Kubernetes Engine
5 participants