adoptium / openj9-systemtest

Long running J9 tests
Other
6 stars 38 forks source link

Accept 99% soft full cache in SharedClassesWorkloadTest_Softmx_Increase test #94

Closed Mesbah-Alam closed 5 years ago

Mesbah-Alam commented 5 years ago

Resolves https://github.com/eclipse/openj9-systemtest/issues/92 Signed-off-by: Mesbah_Alam@ca.ibm.com Mesbah_Alam@ca.ibm.com

FYI @hangshao0

Mesbah-Alam commented 5 years ago

Tested internally: /Test_grinder/job/Grinder_Advanced/965/console

Mesbah-Alam commented 5 years ago

Reviewer @smlambert

Mesbah-Alam commented 5 years ago

Spoke with @hangshao0 and we have agreed to accept a value of 99% soft full for caches as the last entry to be stored to the shared cache might be > 1% of the cache size -- effectively meaning 100% and 99% are both acceptable. This PR updates the test to accept 99% soft-full as well as 100% soft-full after step 2 and step 4 completes.