Update tests to add a sleep before destroying nonpersistent cache - #23881
Conversation
Issue eclipse-openj9#23288 (comment) Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
| <output type="failure" caseSensitive="yes" regex="no">Processing dump event</output> | ||
| </test> | ||
|
|
||
| <exec command="sleep 0.5" platforms="linux.*" /> |
There was a problem hiding this comment.
There are many other places that destroy caches (some in this file): Why was a delay not added before each of those operations?
There was a problem hiding this comment.
It's only deleting nonpersistent caches which is a problem. I only added the delay in the places that were failing in nightly testing. If I see more failures I can add more sleeps. We don't need to slow down the testing unnecessarily by adding sleeps to places that don't need them.
There was a problem hiding this comment.
There may be other test suites which are failing, but these are two main ones. I'll be looking at other failures in the coming days.
There was a problem hiding this comment.
I was referring specifically to "deleting nonpersistent caches" elsewhere. It's a little disconcerting that we don't appear to understand why those other places don't need to change.
Fix another failure in testSCCMLSnapshot which occurred once in the last nightly testing. Related to #23881 Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Fix another failure in testSCCMLSnapshot which occurred once in the last nightly testing. Related to eclipse-openj9#23881 Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
2 participants
Issue #23288 (comment)
Tested in grinders.
testSCCMLTests1_openj9
aarch64 https://openj9-jenkins.osuosl.org/job/Grinder/5174
ppc64le https://openj9-jenkins.osuosl.org/job/Grinder/5176
testSCCMLSnapshot
ppc64le https://openj9-jenkins.osuosl.org/job/Grinder/5178