multus-cni icon indicating copy to clipboard operation
multus-cni copied to clipboard

Added Multus Conf Dir to AutoConfig

Open naeemarsalan opened this issue 2 years ago • 2 comments

Feature:

  • Added cli arg to pass "confDir" to autoconfig

This is required when using distro like RKE2/K3s which has different path for cni like multus. These distro use path: /etc/cni/net.d/ and currently with autoconfig mode uses "/etc/cni/multus/".

The current fix for different is to use a sym link with autoconfig mode. ln -s /etc/cni/net.d/ /etc/cni/multus/

Signed-off-by: [email protected]

naeemarsalan avatar Jun 14 '22 10:06 naeemarsalan

First, thank you for the PR, @naeemarsalan !

The current fix for different is to use a sym link with autoconfig mode. ln -s /etc/cni/net.d/ /etc/cni/multus/

I'm wondering why you mentioned such symbolic link because multus-cni (entrypoint.sh) does not do such things. Is there some deployments that does ln -s /etc/cni/net.d/ /etc/cni/multus/?

s1061123 avatar Jun 15 '22 15:06 s1061123

First, thank you for the PR, @naeemarsalan !

The current fix for different is to use a sym link with autoconfig mode. ln -s /etc/cni/net.d/ /etc/cni/multus/

I'm wondering why you mentioned such symbolic link because multus-cni (entrypoint.sh) does not do such things. Is there some deployments that does ln -s /etc/cni/net.d/ /etc/cni/multus/?

This was a suggested fix if someone else came across the same problem. When using Multus with different K8s (k3s/rke2) this can be quite frustrating, as you are not sure where "/etc/cni/multus/" is coming from.

naeemarsalan avatar Jun 20 '22 09:06 naeemarsalan

This pull request is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days.

github-actions[bot] avatar Sep 19 '22 04:09 github-actions[bot]