-
Notifications
You must be signed in to change notification settings - Fork 722
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
Merge compressedrefs and non-compressedrefs tests builds into one #9231
Comments
fyi @gacholio - for awareness |
Issue: eclipse-openj9/openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
- remove obsolete *_sp, *_hrt, *_purec, *_codecov specs - clean up ottawa.csv and prepare for eclipse-openj9/openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
As suggested by @pshipton , we will update NoOptions to Mode110 and Mode610 in system tests to cover all cases in mixedref SDK. (Remove the duplicates if it is in special.system) Also, run enable parallel in system test builds. For sanity and extended, parallel on 2 machines. Switch special.system test builds from subdir to dynamic parallel on 5 machines. |
- for generic tests for all JVM, add Mode150 and Mode650 - for tests for openj9/ibm only, add Mode110 and Mode610 - remove Mode110 and Mode610 in special tests if above modes are added in sanity/extended Related: eclipse-openj9/openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
- for generic tests for all JVM, add Mode150 and Mode650 - for tests for openj9/ibm only, add Mode110 and Mode610 - remove Mode110 and Mode610 in special tests if above modes are added in sanity/extended Related: eclipse-openj9/openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
- for generic tests for all JVM, add Mode150 and Mode650 - for tests for openj9/ibm only, add Mode110 and Mode610 - remove Mode110 and Mode610 in special tests if above modes are added in sanity/extended Related: eclipse-openj9/openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
vm.cmprssptrs does not apply to mixref SDK. Remove vm.cmprssptrs and use Mode110/610 or Mode150/650 instead [ci skip] Related:eclipse-openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
vm.cmprssptrs does not apply to mixref SDK. Remove vm.cmprssptrs and use Mode110/610 or Mode150/650 instead [ci skip] Related:eclipse-openj9#9231 Signed-off-by: lanxia <lan_xia@ca.ibm.com>
This issue is completed |
Related to the work of combining compressedrefs and non-compressedrefs SPECS into the same build, we are planning to merge two tests builds into one (removing _xl). Below are some todos:
No jvm option will be added for
NoOptions
. Remove the logic in TKG that adds jvm option -Xcompressedrefs or -Xnocompressedrefshttps://github.com/AdoptOpenJDK/TKG/blob/6ad0667a194152ba7a5a98d68b39731d6c2922a3/openj9Settings.mk#L23-L29
Update ottawa.csv to merge compressedrefs and non-compressedrefs into one SPEC.
And clean up ottawa.csv if possible (not required for this issue)
Check to see if we can use the same ottawa.csv for internal vmfarm build
FYI @jdekonin @AdamBrousseau @pshipton @smlambert @DanHeidinga
The text was updated successfully, but these errors were encountered: