Skip to content

Commit 9e6910f

Browse files
committed
trt-2246: lower parallelism for network tests
1 parent 8852d04 commit 9e6910f

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

pkg/test/ginkgo/cmd_runsuite.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -441,14 +441,14 @@ func (o *GinkgoRunSuiteOptions) Run(suite *TestSuite, clusterConfig *clusterdisc
441441
return strings.Contains(t.name, "[sig-network]")
442442
})
443443

444-
buildsTests, openshiftTests := splitTests(openshiftTests, func(t *testCase) bool {
445-
return strings.Contains(t.name, "[sig-builds]")
446-
})
447-
448444
networkTests, openshiftTests := splitTests(openshiftTests, func(t *testCase) bool {
449445
return strings.Contains(t.name, "[sig-network]")
450446
})
451447

448+
buildsTests, openshiftTests := splitTests(openshiftTests, func(t *testCase) bool {
449+
return strings.Contains(t.name, "[sig-builds]")
450+
})
451+
452452
mustGatherTests, openshiftTests := splitTests(openshiftTests, func(t *testCase) bool {
453453
return strings.Contains(t.name, "[sig-cli] oc adm must-gather")
454454
})
@@ -513,11 +513,11 @@ func (o *GinkgoRunSuiteOptions) Run(suite *TestSuite, clusterConfig *clusterdisc
513513
tests = append(tests, storageTestsCopy...)
514514

515515
networkK8sTestsCopy := copyTests(networkK8sTests)
516-
q.Execute(testCtx, networkK8sTestsCopy, max(1, parallelism), testOutputConfig, abortFn) // run network tests separately.
516+
q.Execute(testCtx, networkK8sTestsCopy, max(1, parallelism/2), testOutputConfig, abortFn) // run network tests separately.
517517
tests = append(tests, networkK8sTestsCopy...)
518518

519519
networkTestsCopy := copyTests(networkTests)
520-
q.Execute(testCtx, networkTestsCopy, max(1, parallelism), testOutputConfig, abortFn) // run network tests separately.
520+
q.Execute(testCtx, networkTestsCopy, max(1, parallelism/2), testOutputConfig, abortFn) // run network tests separately.
521521
tests = append(tests, networkTestsCopy...)
522522

523523
buildsTestsCopy := copyTests(buildsTests)
@@ -883,7 +883,7 @@ func determineEnvironmentFlags(ctx context.Context, upgrade bool, dryRun bool) (
883883
envFlagBuilder.AddTopology(&singleReplicaTopology)
884884
}
885885

886-
//Additional flags can only be determined if we are able to obtain the clusterState
886+
// Additional flags can only be determined if we are able to obtain the clusterState
887887
if clusterState != nil {
888888
envFlagBuilder.AddAPIGroups(clusterState.APIGroups.UnsortedList()...).
889889
AddFeatureGates(clusterState.EnabledFeatureGates.UnsortedList()...)
@@ -896,7 +896,7 @@ func determineEnvironmentFlags(ctx context.Context, upgrade bool, dryRun bool) (
896896

897897
arch := "Unknown"
898898
if len(clusterState.Masters.Items) > 0 {
899-
//TODO(sgoeddel): eventually, we may need to check every node and pass "multi" as the value if any of them differ from the masters
899+
// TODO(sgoeddel): eventually, we may need to check every node and pass "multi" as the value if any of them differ from the masters
900900
arch = clusterState.Masters.Items[0].Status.NodeInfo.Architecture
901901
}
902902
envFlagBuilder.AddArchitecture(arch)

0 commit comments

Comments
 (0)