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

Link to linux troubleshoot #13529

Merged
merged 2 commits into from Mar 2, 2022
Merged

Link to linux troubleshoot #13529

merged 2 commits into from Mar 2, 2022

Conversation

NHingerl
Copy link
Contributor

@NHingerl NHingerl commented Mar 2, 2022

Description

Changes proposed in this pull request:

  • link linux Kyma installation to troubleshooting guide
  • ...
  • ...

Related issue(s)

#13489

@NHingerl NHingerl self-assigned this Mar 2, 2022
@NHingerl NHingerl added the area/documentation Issues or PRs related to documentation label Mar 2, 2022
@kyma-bot kyma-bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Mar 2, 2022
@netlify
Copy link

netlify bot commented Mar 2, 2022

✔️ 🥰 Documentation preview ready! 🥰

🔨 Explore the source changes: 67b0486

🔍 Inspect the deploy log: https://app.netlify.com/sites/kyma-project-docs-preview/deploys/621f3ce06e97dd0007eafe11

😎 Browse the preview: https://deploy-preview-13529--kyma-project-docs-preview.netlify.app

@kyma-bot kyma-bot added the lgtm Looks good to me! label Mar 2, 2022
> ```bash
> kyma provision k3d --name='{CUSTOM_NAME}' --k3s-arg='--alsologtostderr@server:0'
> ```
But you can do more. To define the name of your k3d cluster and pass arguments to the Kubernetes API server, for example, to log to stderr, run:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
But you can do more. To define the name of your k3d cluster and pass arguments to the Kubernetes API server, for example, to log to stderr, run:
To define the name of your k3d cluster and pass arguments to the Kubernetes API server, for example, to log to stderr, run:

I'm not sure if we need the first sentence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree we don't strictly need it, but the whole section is so dominated by boxes (code boxes, Note box) that a little more flesh on the normal text made it easier to read for me.

image

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, now I see it :)

@kyma-bot kyma-bot merged commit c63dbea into kyma-project:main Mar 2, 2022
@NHingerl NHingerl deleted the link_to_linux_troubleshoot branch March 16, 2022 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Issues or PRs related to documentation lgtm Looks good to me! size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants