-
Notifications
You must be signed in to change notification settings - Fork 3.4k
docs: adds a rancher kubernetes engine installation guide #15323
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
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!
Some suggestions below, but they're not blocking.
41639aa
to
a4a48fb
Compare
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 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>
a4a48fb
to
ae68c89
Compare
@joestringer I simply missed adding the file, re-pushed |
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.
Ah, that explains the mixup. LGTM, thanks!
Signed-off-by: Sean Winn sean@isovalent.com
Adds a getting started guide for Rancher Kubernetes Engine
Fixes: #11380