From 2d533342119dc7265f67888efd39a24711f8d9ab Mon Sep 17 00:00:00 2001 From: Tomofumi Hayashi Date: Tue, 5 Apr 2022 02:03:33 +0900 Subject: [PATCH] Remove error handling for getPod to force to proceed cmdDel. In cmdDel, CNI Spec mentioned that plugin should proceed cmdDel without any error, hence the change removes error returning at cmdDel. fix #822 --- pkg/multus/multus.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/multus/multus.go b/pkg/multus/multus.go index 83ba8349f..b60bc69c0 100644 --- a/pkg/multus/multus.go +++ b/pkg/multus/multus.go @@ -815,7 +815,8 @@ func CmdDel(args *skel.CmdArgs, exec invoke.Exec, kubeClient *k8s.ClientInfo) er pod, err := getPod(kubeClient, k8sArgs, true) if err != nil { - return err + // getPod may be failed but just do print error in its log and continue to delete + logging.Errorf("Multus: getPod failed: %v, but continue to delete", err) } // Read the cache to get delegates json for the pod