-
-
Notifications
You must be signed in to change notification settings - Fork 306
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
Rerun in Grinder with failed test targets link does not capture all params #4049
Comments
"Rerun in Grinder with failed test targets:" returns incorrect TESTLIST. Observed in: https://hyc-runtimes-jenkins.swg-devops.com/job/Test_openjdk11_j9_sanity.functional_aarch64_linux/285/. current output: expected output example: |
re #4049 (comment), it is related to TKG_ITERATIONS. @renfeiw could you please take a look? Thanks |
Now that TKG#373 is merged we should verify that this issue can be closed. |
It generated the correct target in my grinder: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/6208/ @momanmalik Could you please verify if this solves your issue? |
@renfeiw The issue appears to be resolved on my end. |
Describe the bug
Rerun in Grinder with failed test targets link on test job does not capture all parameters info. For example, in https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.openjdk_x86-64_mac/96/ , Rerun in Grinder with failed test targets link only has params for SDK_RESOURCE and TARGET :
https://ci.adoptopenjdk.net/job/Grinder/parambuild/?SDK_RESOURCE=upstream&TARGET=testList+TESTLIST=jvm_compiler_0%20,jdk_net_1
It does not have all the params info and it will cause failure in Grinder: https://ci.adoptopenjdk.net/job/Grinder/5790/console
Also, there is a space between tests in testList, which should be removed.
Expected behavior
It should capture all params same as Rerun in Grinder
Screenshots
![image](https://user-images.githubusercontent.com/19273206/196476100-5b3795b3-2def-4968-8a57-d0a338b4caa2.png)
The text was updated successfully, but these errors were encountered: