revert #33218. dont need #36180. We only use diskpressure

plugin/pkg/scheduler/algorithm/predicates/error.go

@@ -37,7 +37,6 @@ var (
 	ErrMaxVolumeCountExceeded    = newPredicateFailureError("MaxVolumeCount")
 	ErrNodeUnderMemoryPressure   = newPredicateFailureError("NodeUnderMemoryPressure")
 	ErrNodeUnderDiskPressure     = newPredicateFailureError("NodeUnderDiskPressure")
-	ErrNodeUnderInodePressure    = newPredicateFailureError("NodeUnderInodePressure")
 	// ErrFakePredicate is used for test only. The fake predicates returning false also returns error
 	// as ErrFakePredicate.
 	ErrFakePredicate = newPredicateFailureError("FakePredicateError")

plugin/pkg/scheduler/algorithm/predicates/predicates.go

@@ -1219,21 +1219,3 @@ func CheckNodeDiskPressurePredicate(pod *api.Pod, meta interface{}, nodeInfo *sc
 
 	return true, nil, nil
 }
-
-// CheckNodeInodePressurePredicate checks if a pod can be scheduled on a node
-// reporting inode pressure condition.
-func CheckNodeInodePressurePredicate(pod *api.Pod, meta interface{}, nodeInfo *schedulercache.NodeInfo) (bool, []algorithm.PredicateFailureReason, error) {
-	node := nodeInfo.Node()
-	if node == nil {
-		return false, nil, fmt.Errorf("node not found")
-	}
-
-	// is node under presure?
-	for _, cond := range node.Status.Conditions {
-		if cond.Type == api.NodeInodePressure && cond.Status == api.ConditionTrue {
-			return false, []algorithm.PredicateFailureReason{ErrNodeUnderInodePressure}, nil
-		}
-	}
-
-	return true, nil, nil
-}

plugin/pkg/scheduler/algorithm/predicates/predicates_test.go

@@ -3142,75 +3142,3 @@ func TestPodSchedulesOnNodeWithDiskPressureCondition(t *testing.T) {
 		}
 	}
 }
-
-func TestPodSchedulesOnNodeWithInodePressureCondition(t *testing.T) {
-	pod := &api.Pod{
-		Spec: api.PodSpec{
-			Containers: []api.Container{
-				{
-					Name:            "container",
-					Image:           "image",
-					ImagePullPolicy: "Always",
-				},
-			},
-		},
-	}
-
-	// specify a node with no inode pressure condition on
-	noPressureNode := &api.Node{
-		Status: api.NodeStatus{
-			Conditions: []api.NodeCondition{
-				{
-					Type:   api.NodeReady,
-					Status: api.ConditionTrue,
-				},
-			},
-		},
-	}
-
-	// specify a node with pressure condition on
-	pressureNode := &api.Node{
-		Status: api.NodeStatus{
-			Conditions: []api.NodeCondition{
-				{
-					Type:   api.NodeInodePressure,
-					Status: api.ConditionTrue,
-				},
-			},
-		},
-	}
-
-	tests := []struct {
-		pod      *api.Pod
-		nodeInfo *schedulercache.NodeInfo
-		fits     bool
-		name     string
-	}{
-		{
-			pod:      pod,
-			nodeInfo: makeEmptyNodeInfo(noPressureNode),
-			fits:     true,
-			name:     "pod schedulable on node without inode pressure condition on",
-		},
-		{
-			pod:      pod,
-			nodeInfo: makeEmptyNodeInfo(pressureNode),
-			fits:     false,
-			name:     "pod not schedulable on node with inode pressure condition on",
-		},
-	}
-	expectedFailureReasons := []algorithm.PredicateFailureReason{ErrNodeUnderInodePressure}
-
-	for _, test := range tests {
-		fits, reasons, err := CheckNodeInodePressurePredicate(test.pod, PredicateMetadata(test.pod, nil), test.nodeInfo)
-		if err != nil {
-			t.Errorf("%s: unexpected error: %v", test.name, err)
-		}
-		if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
-			t.Errorf("%s: unexpected failure reasons: %v, want: %v", test.name, reasons, expectedFailureReasons)
-		}
-		if fits != test.fits {
-			t.Errorf("%s: expected %v got %v", test.name, test.fits, fits)
-		}
-	}
-}