Fixed an issue where pub webhooks unexpectedly return error when PUB …

…is NOT FOUND (#1579) * Fixed an issue where pub webhooks unexpectedly return error when PUB is NOT FOUND Signed-off-by: JinXinWang <[email protected]> * Fixed an issue where pub webhooks unexpectedly return error when PUB is NOT FOUND Signed-off-by: JinXinWang <[email protected]> * Fixed an issue where pub webhooks unexpectedly return error when PUB is NOT FOUND Signed-off-by: JinXinWang <[email protected]> --------- Signed-off-by: JinXinWang <[email protected]>
openkruise · Apr 23, 2024 · 2da1b90 · 2da1b90
1 parent 38f432f
commit 2da1b90
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/pkg/control/pubcontrol/pub_control_utils.go b/pkg/control/pubcontrol/pub_control_utils.go
@@ -103,6 +103,9 @@ func PodUnavailableBudgetValidatePod(pod *corev1.Pod, operation policyv1alpha1.P
 			pubClone, err = kubeClient.GetGenericClient().KruiseClient.PolicyV1alpha1().
 				PodUnavailableBudgets(pub.Namespace).Get(context.TODO(), pub.Name, metav1.GetOptions{})
 			if err != nil {
+				if errors.IsNotFound(err) {
+					return nil
+				}
 				klog.ErrorS(err, "Failed to get podUnavailableBudget form etcd", "pub", klog.KObj(pub))
 				return err
 			}
@@ -169,6 +172,10 @@ func PodUnavailableBudgetValidatePod(pod *corev1.Pod, operation policyv1alpha1.P
 				klog.ErrorS(err, "Failed to add cache for podUnavailableBudget", "pub", klog.KObj(pub))
 			}
 			return nil
+		} else {
+			if errors.IsNotFound(err) {
+				return nil
+			}
 		}
 		// if conflicts, then retry
 		conflictTimes++