-
Notifications
You must be signed in to change notification settings - Fork 178
Clarify steps and add drop-down menu #463
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
c1ca46f to
0b4ef5f
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.
We should add additional information to the summaries. With the sections collapsed by default, users will miss critical information that might tell them whether or not they need to perform these steps.
docs/installation/airgap.md
Outdated
|
|
||
| When running the K3s script with the `INSTALL_K3S_SKIP_DOWNLOAD` environment variable, K3s will use the local version of the script and binary. | ||
| <details> | ||
| <summary>**Set Default Network Route**</summary> |
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.
| <summary>**Set Default Network Route**</summary> | |
| <summary>**Set Default Network Route** - required for nodes without a default route</summary> |
docs/installation/airgap.md
Outdated
|
|
||
| #### SELinux RPM | ||
| <details> | ||
| <summary>**Download SELinux RPM**</summary> |
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.
| <summary>**Download SELinux RPM**</summary> | |
| <summary>**Download SELinux RPM** - required for airgapped nodes with SELinux enabled</summary> |
Signed-off-by: manuelbuil <[email protected]>
0b4ef5f to
8aad405
Compare
Clarify some steps in the manually deploy images section and add a drop-down menu:

and