k8snetworkplumbingwg / multus-cni / 8830351525

63%

master: 56%

Build Type

Pull #1270

github

Committed by ordovicia

Commit Message

thin plugin: Handle --multus-master-cni-file-name flag

Multus v3.9.3 has `--multus-master-cni-file-name` flag to specify the
name of a primary CNI config file.
https://github.com/k8snetworkplumbingwg/multus-cni/blob/v3.9.3/images/entrypoint.sh#L22

In Multus v4.0.2, the thin plugin has the flag defined, but it is not
read and so does not have effect.

This pull request fixes the problem by making the thin plugin correctly
handles `--multus-master-cni-file-name` flag.

Fixes #1226

Signed-off-by: Hidehito Yabuuchi <hyab@preferred.jp>

Pull Request Pull Request #1270: thin plugin: Handle `--multus-master-cni-file-name` flag

Run Details

9 of 12 new or added lines in 1 file covered. (75.0%)

36 existing lines in 3 files now uncovered.

2273 of 3605 relevant lines covered (63.05%)

16.34 hits per line