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: de-duplicate instructions in kind and hubble getting started guides #12210
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.
lgtm, a couple of non-blocking minor nits
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, thanks for tidying this up.
A made some minor suggestions below to use imperative sense to simplify the language on the steps necessary to install Helm. (I realise they're not all from this PR, but I think that they clarify the wording).
I also made some suggestions on the use of sphinx directives for syntax highlighting, things like .. parsed-literal
usage when literals are not passed.
ad67f34
to
1b564fc
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.
LGTM, looks like the CI is just complaining on one minor spelling point because it doesn't know the word Preload
.
These 2 guides have a lot of instructions in common as the Hubble guide instructs to create a sandbox environment using Kind. This commit ensures that instructions are not duplicated in the two guides but are included from separate files from the 2 guides. In addition to this, this commit also updates the instructions about possible conflict of IP address when deploying a cluster with Kind: the "tip" has been moved to the "Configure Kind" section as the fix needs to be applied when configuring Kind. Moreover, remove the "Optional" wording with regard to pre-loading the cilium image into the Kind cluster. Signed-off-by: Robin Hahling <robin.hahling@gw-computing.net>
1b564fc
to
f4d7847
Compare
These 2 guides have a lot of instructions in common as the Hubble guide
instructs to create a sandbox environment using Kind. This commit
ensures that instructions are not duplicated in the two guides but are
included from separate files from the 2 guides.
In addition to this, this commit also updates the instructions about
possible conflict of IP address when deploying a cluster with Kind: the
"tip" has been moved to the "Configure Kind" section as the fix needs to
be applied when configuring Kind.
Moreover, remove the "Optional" wording with regard to pre-loading the
cilium image into the Kind cluster.