Fix Intermittent vlog not found CRIU Jitserver test failures - #17578
Conversation
3364a53 to
cdc2704
Compare
|
jenkins test sanity.functional xlinux jdk17 |
Well... clearly this didn't fix it... I think I'll update the test to cat the EDIT: Technically this is a different problem; if the vlog didnt exist then we'd enter the |
cdc2704 to
367bd7b
Compare
|
jenkins test sanity.functional xlinux jdk17 |
367bd7b to
597e9af
Compare
|
jenkins test sanity.functional xlinux jdk17 |
|
Launched a 50x grinder here https://openj9-jenkins.osuosl.org/job/Grinder/2506/; lets see if we can reproduce the failure. |
597e9af to
61d947d
Compare
61d947d to
c4640eb
Compare
|
A thought occurred to me just now; if the restore fails in one of the known ways, is that going to show up in |
Signed-off-by: Irwin D'Souza <dsouzai.gh@gmail.com>
c4640eb to
fd6cb59
Compare
|
I updated the scripts to save the output of the |
|
jenkins test sanity plinux,xlinux,zlinux jdk17 |
|
Failure on zlinux: cmdLineTester_criu_nonPortableRestore_0 Another (probably unrelated) failure on plinux: cmdLineTest_J9test_common_0 |
|
I think the zlinux failure is the same as #16980. The plinux failure can't be related because it's in a completely different test that I haven't modified. |
|
Since the two failures are unrelated, this PR is good for merging |
Sometimes, a restore can fail in a known way, in which case the test is set to pass; however, this causes subsequent tests that depend on the output of this test to fail. This PR adds a new script to wrap the
catcommands that test for certain Jitserver characteristics post restore. This wrapper script will check the output of the previous test if the vlog to cat does not exist, and outputs a message that causes the test to pass if the previous test failed in one of the known ways.This PR also modifies the existing criu scripts to take in an additional parameter to not delete the
testOutputfile if subsequent tests need it.Fixes #17367