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

Multus page updates #336

Open
nwneisen opened this issue Mar 21, 2025 · 0 comments · May be fixed by #337 or #338
Open

Multus page updates #336

nwneisen opened this issue Mar 21, 2025 · 0 comments · May be fixed by #337 or #338
Assignees
Labels

Comments

@nwneisen
Copy link
Collaborator

Found while taking a look at our Multus setup:

  • We have some pages that assume mke4.yaml for the config file when running an apply. Some don't have anything. This page uses <mke-configuration-file>, it should be changed to whatever is consistent.
  • Enable Multus -> 4 says MetalLb when this is Multus.
  • Add a network interface should start with a step that you need to SSH in to each node to run the commands. It is currently easy to miss.
  • This command is formatted incorrectly. Once it is copied, either manually or using the button, it is invalid bash
CNI_PLUGIN_VERSION=v1.3.0
CNI_ARCH=amd64
curl -sL
https://github.com/containernetworking/plugins/releases/download/${CNI_PLUGIN_VERSION}/cni-plugins-linux-${CNI_ARCH}-${CNI_PLUGIN_VERSION}.tgz
| sudo tar xvz -C /opt/cni/bin/

It should be

CNI_PLUGIN_VERSION=v1.3.0
CNI_ARCH=amd64
curl -sL https://github.com/containernetworking/plugins/releases/download/${CNI_PLUGIN_VERSION}/cni-plugins linux-${CNI_ARCH}-${CNI_PLUGIN_VERSION}.tgz | sudo tar xvz -C /opt/cni/bin/

In which the curl command is all on one line.

If it has to be on multiple lines, then \ should be used.

CNI_PLUGIN_VERSION=v1.3.0
CNI_ARCH=amd64
curl -sL \
https://github.com/containernetworking/plugins/releases/download/${CNI_PLUGIN_VERSION}/cni-plugins-linux-${CNI_ARCH}-${CNI_PLUGIN_VERSION}.tgz \
| sudo tar xvz -C /opt/cni/bin/
  • Step 4 should separate the command from the output so that it can be easily copied
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants