Merge pull request #29500 from lixiaobing10051267/masterFound

Automatic merge from submit-queue

Check all places to break the loop when object found

Check all places to break the loop when object found.

pkg/apis/rbac/validation/policy_comparator_test.go

@@ -389,6 +389,7 @@ func rulesMatch(expectedRules, actualRules []rbac.PolicyRule) bool {
 		for _, actualRule := range actualRules {
 			if reflect.DeepEqual(expectedRule, actualRule) {
 				found = true
+				break
 			}
 		}
 

pkg/client/testing/core/fixture.go

@@ -350,6 +350,7 @@ func (t *tracker) Delete(gvk unversioned.GroupVersionKind, ns, name string) erro
 		if objMeta.GetNamespace() == ns && objMeta.GetName() == name {
 			t.objects[gvk] = append(t.objects[gvk][:i], t.objects[gvk][i+1:]...)
 			found = true
+			break
 		}
 	}
 

pkg/controller/petset/pet_set_test.go

@@ -219,6 +219,7 @@ func TestPetSetControllerBlocksScaling(t *testing.T) {
 	for _, p := range fc.getPodList() {
 		if p.Name == deletedPod.Name {
 			found = true
+			break
 		}
 	}
 	if !found {

pkg/kubectl/resource/builder_test.go

@@ -445,6 +445,7 @@ func TestDirectoryBuilder(t *testing.T) {
 	for _, info := range test.Infos {
 		if info.Name == "redis-master" && info.Namespace == "test" && info.Object != nil {
 			found = true
+			break
 		}
 	}
 	if !found {

pkg/kubelet/dockertools/docker_manager_test.go

@@ -624,6 +624,7 @@ func TestSyncPodCreateNetAndContainer(t *testing.T) {
 	for _, c := range fakeDocker.RunningContainerList {
 		if c.Image == "pod_infra_image" && strings.HasPrefix(c.Names[0], "/k8s_POD") {
 			found = true
+			break
 		}
 	}
 	if !found {