fix: Avoid retrying caused by 404 when deleting cache (#424)

Author: gxthrj

pkg/apisix/global_rule.go

@@ -187,7 +187,9 @@ func (r *globalRuleClient) Delete(ctx context.Context, obj *v1.GlobalRule) error
 	}
 	if err := r.cluster.cache.DeleteGlobalRule(obj); err != nil {
 		log.Errorf("failed to reflect global_rule delete to cache: %s", err)
-		return err
+		if err != cache.ErrNotFound {
+			return err
+		}
 	}
 	return nil
 }

pkg/apisix/route.go

@@ -189,7 +189,9 @@ func (r *routeClient) Delete(ctx context.Context, obj *v1.Route) error {
 	}
 	if err := r.cluster.cache.DeleteRoute(obj); err != nil {
 		log.Errorf("failed to reflect route delete to cache: %s", err)
-		return err
+		if err != cache.ErrNotFound {
+			return err
+		}
 	}
 	return nil
 }

pkg/apisix/ssl.go

@@ -185,7 +185,9 @@ func (s *sslClient) Delete(ctx context.Context, obj *v1.Ssl) error {
 	}
 	if err := s.cluster.cache.DeleteSSL(obj); err != nil {
 		log.Errorf("failed to reflect ssl delete to cache: %s", err)
-		return err
+		if err != cache.ErrNotFound {
+			return err
+		}
 	}
 	return nil
 }

pkg/apisix/stream_route.go

@@ -186,7 +186,9 @@ func (r *streamRouteClient) Delete(ctx context.Context, obj *v1.StreamRoute) err
 	}
 	if err := r.cluster.cache.DeleteStreamRoute(obj); err != nil {
 		log.Errorf("failed to reflect stream_route delete to cache: %s", err)
-		return err
+		if err != cache.ErrNotFound {
+			return err
+		}
 	}
 	return nil
 }

pkg/apisix/upstream.go

@@ -182,7 +182,9 @@ func (u *upstreamClient) Delete(ctx context.Context, obj *v1.Upstream) error {
 	}
 	if err := u.cluster.cache.DeleteUpstream(obj); err != nil {
 		log.Errorf("failed to reflect upstream delete to cache: %s", err.Error())
-		return err
+		if err != cache.ErrNotFound {
+			return err
+		}
 	}
 	return nil
 }