Merge pull request #67 from almaslennikov/success-missing-config

Return success if config is not found on DEL action
k8snetworkplumbingwg · Jan 23, 2023 · 848e3b4 · 848e3b4
2 parents 3f44c9e + c3d3fc6
commit 848e3b4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/cmd/ib-sriov-cni/main.go b/cmd/ib-sriov-cni/main.go
@@ -298,7 +298,10 @@ func cmdDel(args *skel.CmdArgs) error {
 
 	netConf, cRefPath, err := config.LoadConfFromCache(args)
 	if err != nil {
-		return err
+		// According to the CNI spec, a DEL action should complete without errors
+		// even if there are some resources missing. For more details, see
+		// https://github.com/containernetworking/cni/blob/main/SPEC.md#del-remove-container-from-network-or-un-apply-modifications
+		return nil
 	}
 
 	defer func() {