Skip to content

Commit

Permalink
Suppress error message in cmdDel, in thick plugin
Browse files Browse the repository at this point in the history
This fix suppress error message in thick plugin's shim, for
DEL command, to align with CNI spec.

Fix k8snetworkplumbingwg#1080
  • Loading branch information
s1061123 committed May 2, 2023
1 parent b05ff2d commit 5d64ec3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/server/api/shim.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,8 @@ func CmdCheck(args *skel.CmdArgs) error {
func CmdDel(args *skel.CmdArgs) error {
_, _, err := postRequest(args)
if err != nil {
return logging.Errorf("CmdDel (shim): %v", err)
// No error in DEL (as of CNI spec)
logging.Errorf("CmdCheck (shim): %v", err)
}
return nil
}
Expand Down

0 comments on commit 5d64ec3

Please sign in to comment.