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

Support modifying the CNI configuration file name

Open lujinda opened this issue 1 year ago • 4 comments

lujinda avatar Jul 22 '22 01:07 lujinda

Coverage Status

Coverage decreased (-0.07%) to 70.284% when pulling bec3b159ae375fdf6016e6586460fc737c63bddb on lujinda:master into 70c970cd6e543cffc043728558652f7402364b28 on k8snetworkplumbingwg:master.

coveralls avatar Jul 22 '22 05:07 coveralls

I'm not clear that why we need the option. Could you please let me know the usecase?

s1061123 avatar Jul 22 '22 10:07 s1061123

I'm not clear that why we need the option. Could you please let me know the usecase?

Thank you for your reply. In our scenario, we cannot assume that 00-multus.conf is the first configuration. In some of our scenarios, the first configuration can be named 00-cni.conf

lujinda avatar Jul 25 '22 01:07 lujinda

We'll need an update to the configuration docs as well @ https://github.com/k8snetworkplumbingwg/multus-cni/blob/master/docs/configuration.md

dougbtv avatar Jul 26 '22 15:07 dougbtv

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 Nov 17 '22 02:11 github-actions[bot]