Commit 4b5c74ee authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub

Merge pull request #31859 from resouer/fix-nits

Automatic merge from submit-queue Fix minor nits in test cases Found a group of nits when doing #30844, fixed them in a this PR since 30844 requires a long time to review.
parents a06123c0 608e7ce0
...@@ -747,7 +747,7 @@ func TestPodFitsSelector(t *testing.T) { ...@@ -747,7 +747,7 @@ func TestPodFitsSelector(t *testing.T) {
}, },
}, },
labels: map[string]string{ labels: map[string]string{
// We use two digit to denote major version and two digit for minior version. // We use two digit to denote major version and two digit for minor version.
"kernel-version": "0206", "kernel-version": "0206",
}, },
fits: true, fits: true,
...@@ -2619,7 +2619,7 @@ func TestPodToleratesTaints(t *testing.T) { ...@@ -2619,7 +2619,7 @@ func TestPodToleratesTaints(t *testing.T) {
}, },
}, },
fits: true, fits: true,
test: "a pod which can be scheduled on a dedicated node assgined to user1 with effect NoSchedule", test: "a pod which can be scheduled on a dedicated node assigned to user1 with effect NoSchedule",
}, },
{ {
pod: &api.Pod{ pod: &api.Pod{
...@@ -2652,7 +2652,7 @@ func TestPodToleratesTaints(t *testing.T) { ...@@ -2652,7 +2652,7 @@ func TestPodToleratesTaints(t *testing.T) {
}, },
}, },
fits: false, fits: false,
test: "a pod which can't be scheduled on a dedicated node assgined to user2 with effect NoSchedule", test: "a pod which can't be scheduled on a dedicated node assigned to user2 with effect NoSchedule",
}, },
{ {
pod: &api.Pod{ pod: &api.Pod{
...@@ -2826,7 +2826,7 @@ func TestPodToleratesTaints(t *testing.T) { ...@@ -2826,7 +2826,7 @@ func TestPodToleratesTaints(t *testing.T) {
}, },
fits: true, fits: true,
test: "The pod has a toleration that key and value don't match the taint on the node, " + test: "The pod has a toleration that key and value don't match the taint on the node, " +
"but the effect of taint on node is PreferNochedule. Pod can be shceduled onto the node", "but the effect of taint on node is PreferNochedule. Pod can be scheduled onto the node",
}, },
} }
expectedFailureReasons := []algorithm.PredicateFailureReason{ErrTaintsTolerationsNotMatch} expectedFailureReasons := []algorithm.PredicateFailureReason{ErrTaintsTolerationsNotMatch}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment