Azure / karpenter-provider-azure / 13462788968

94%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add azure-cni-overlay label and set network plugin to none in some cases (#697)

* fix: setting network plugin to none in the agentbaker contract if we are using overlay net plugin mode.
This will delegate the installation of the cni to daemonsets (azure-cns) etc.
This also introduces a label azure-cni-overlay which adds better consistency between karpenter provided nodes and nodes provided from aks.
We also introduce tests to explicitly validate that if the network plugin mode on the cluster is azure and uses azure cni overlay, that we properly delegate that installation

* fix: ci

* refactor: renaming cni label var

* modifying comment

Co-authored-by: Alex Leites <18728999+tallaxes@users.noreply.github.com>

* style: renaming vars to not have vnet prefix

* test: adding tests for the different cni modes on labels

* refactor: renaming test

* feat: supporting cilium + nodesubnet case

---------

Co-authored-by: Alex Leites <18728999+tallaxes@users.noreply.github.com>

Run Details

19 of 19 new or added lines in 2 files covered. (100.0%)

2 existing lines in 1 file now uncovered.

48329 of 50674 relevant lines covered (95.37%)

0.96 hits per line