tmp: Ignore filter denial in method failure metrics

neondatabase · Dec 13, 2024 · c7dee65 · c7dee65
1 parent 6c76e9d
commit c7dee65
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/pkg/plugin/framework_methods.go b/pkg/plugin/framework_methods.go
@@ -96,7 +96,8 @@ func (m frameworkMetrics) incMethodCall(method string, pod *corev1.Pod, ignored
 }
 
 func (m frameworkMetrics) incFailIfnotSuccess(method string, pod *corev1.Pod, ignored bool, status *framework.Status) {
-	if status.IsSuccess() {
+	// it's normal for Filter to return Unschedulable, because that's its way of filtering out pods.
+	if status.IsSuccess() || (method == "Filter" && status.Code() == framework.Unschedulable) {
 		return
 	}