Skip to content
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

Update PARALLEL description in testJobTemplate #2943

Closed
llxia opened this issue Oct 1, 2021 · 7 comments · Fixed by #3429
Closed

Update PARALLEL description in testJobTemplate #2943

llxia opened this issue Oct 1, 2021 · 7 comments · Fixed by #3429

Comments

@llxia
Copy link
Contributor

llxia commented Oct 1, 2021

It would be nice to have PARALLEL description updated as following in testJobTemplate
https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/testJobTemplate#L374

Optional. Parallel mode: <br/>
None // run test(s) in serial <br/>
Dynamic // dynamically divides tests according to NUM_MACHINES and runs them in parallel. Please also provide NUM_MACHINES. <br/>
NodesByIterations // run the same test(s) in parallel according to NUM_MACHINES. You can also provide ITERATIONS. This is mainly for debugging intermittent issues. <br/> 
Subdir // run tests in parallel according to subdir. This is mainly for external tests.
@abigailsleek
Copy link

I would like to work on this.

@smlambert
Copy link
Contributor

hi @abigailsleek - you also asked to be assigned to #2949, do you have a preference between the 2? We will only assign one issue at a time.

@abigailsleek
Copy link

abigailsleek commented Oct 8, 2021 via email

@smlambert
Copy link
Contributor

Very good! thanks @abigailsleek!

@abigailsleek
Copy link

abigailsleek commented Oct 10, 2021 via email

@smlambert
Copy link
Contributor

PR #2973 was closed without being merged (was unable to pass the ECA validation). Un-assigning this and putting it back in the list of open / available good first issues.

@abhijeetjejurkar
Copy link
Contributor

Hey Can I work on this?

abhijeetjejurkar added a commit to abhijeetjejurkar/aqa-tests that referenced this issue Mar 8, 2022
This was Updated in AQAvit v0.8.0
Fixes: adoptium#2943

Signed-off-by: Abhijeet Jejurkar <abhijeet.jejurkar2@gmail.com>
abhijeetjejurkar added a commit to abhijeetjejurkar/aqa-tests that referenced this issue Mar 11, 2022
abhijeetjejurkar added a commit to abhijeetjejurkar/aqa-tests that referenced this issue Mar 18, 2022
Fixes: adoptium#2943

Signed-off-by: Abhijeet Jejurkar <abhijeet.jejurkar2@gmail.com>
smlambert pushed a commit that referenced this issue Mar 22, 2022
* Updated PARALLEL description in testJobTemplate
Fixes: #2943

* Added according to NUM_MACHINES or TEST_TIME on Line-388
Fixes: #2943

Signed-off-by: Abhijeet Jejurkar <abhijeet.jejurkar2@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants