-
-
Notifications
You must be signed in to change notification settings - Fork 304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jenkins: Parallel option should be ignored when num machines is 1 #4093
Comments
I didn't look at the code yet to see how feasible this is. |
If we make this change, we could also change the parallel default in a grinder from "None" to (my preferred) "NodesByIterations". |
llxia
added a commit
to llxia/openjdk-tests
that referenced
this issue
Oct 28, 2022
resolve: adoptium#4093 Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
llxia
added a commit
to llxia/openjdk-tests
that referenced
this issue
Oct 31, 2022
resolve: adoptium#4093 Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
llxia
added a commit
to llxia/openjdk-tests
that referenced
this issue
Nov 1, 2022
resolve: adoptium#4093 Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
llxia
added a commit
to llxia/openjdk-tests
that referenced
this issue
Nov 1, 2022
When running tests with Parallel option and num_machines=1, run tests serially. Don't do parallel as it will create unnecessary overhead resolve: adoptium#4093 Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
smlambert
pushed a commit
that referenced
this issue
Nov 2, 2022
When running tests with Parallel option and num_machines=1, run tests serially. Don't do parallel as it will create unnecessary overhead resolve: #4093 Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com> Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
If the number of machines, and therefore the number of child jobs, equals 1, I don't think we need to bother launching a child job.
To Reproduce
Launch a test with a parallel option and a build node label that only matches 1 machine
Expected behavior
Don't do parallel as it will create unnecessary overhead
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: