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 #1080
  • Loading branch information
s1061123 committed May 2, 2023
1 parent b9acfeb commit 1a4d045
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 1a4d045

Please sign in to comment.