Fix executor failed to clean cache & kubeobjs after function deleted (#534)

executor/cleanup.go

@@ -22,6 +22,7 @@ import (
 	"strings"
 	"time"
 
+	"k8s.io/apimachinery/pkg/api/errors"
 	meta_v1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/labels"
 	"k8s.io/client-go/kubernetes"
@@ -117,18 +118,24 @@ func idleObjectReaper(kubeClient *kubernetes.Clientset,
 			for _, fsvc := range funcSvcs {
 
 				fn, err := fissionClient.Functions(fsvc.Function.Namespace).Get(fsvc.Function.Name)
+				if err == nil {
+					// Ignore functions of NewDeploy ExecutorType with MinScale > 0
+					if fn.Spec.InvokeStrategy.ExecutionStrategy.MinScale > 0 &&
+						fn.Spec.InvokeStrategy.ExecutionStrategy.ExecutorType == fission.ExecutorTypeNewdeploy {
+						continue
+					}
+				}
 
-				if err != nil {
+				// Return errors not equal to "is not found" error
+				if err != nil && !errors.IsNotFound(err) {
 					log.Printf("Error getting function: %v", fsvc.Function.Name)
 					continue
 				}
 
-				// Ignore functions of NewDeploy ExecutorType with MinScale > 0
+				// Newdeploy manager handles the function delete event and clean cache/kubeobjs itself,
-				if fn.Spec.InvokeStrategy.ExecutionStrategy.MinScale > 0 && fn.Spec.InvokeStrategy.ExecutionStrategy.ExecutorType == fission.ExecutorTypeNewdeploy {
+				// so we ignore the function service cache with newdepoy executor type here.
-					continue
+				if fsvc.Executor != fscache.NEWDEPLOY {
-				}
 					deleted, err := fsCache.DeleteOld(fsvc, idlePodReapTime)
-
 					if err != nil {
 						log.Printf("Error deleting Kubernetes objects for fsvc '%v': %v", fsvc, err)
 						log.Printf("Object Name| Object Kind | Object Namespace")
@@ -147,6 +154,7 @@ func idleObjectReaper(kubeClient *kubernetes.Clientset,
 			}
 		}
 	}
+}
 
 func deleteKubeobject(kubeClient *kubernetes.Clientset, kubeobj *api.ObjectReference) {
 	switch strings.ToLower(kubeobj.Kind) {