Skip to content

Commit 848e3b4

Browse files
authored
Merge pull request k8snetworkplumbingwg#67 from almaslennikov/success-missing-config
Return success if config is not found on DEL action
2 parents 3f44c9e + c3d3fc6 commit 848e3b4

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

cmd/ib-sriov-cni/main.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,10 @@ func cmdDel(args *skel.CmdArgs) error {
298298

299299
netConf, cRefPath, err := config.LoadConfFromCache(args)
300300
if err != nil {
301-
return err
301+
// According to the CNI spec, a DEL action should complete without errors
302+
// even if there are some resources missing. For more details, see
303+
// https://github.com/containernetworking/cni/blob/main/SPEC.md#del-remove-container-from-network-or-un-apply-modifications
304+
return nil
302305
}
303306

304307
defer func() {

0 commit comments

Comments
 (0)