-
-
Notifications
You must be signed in to change notification settings - Fork 308
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
Multiple JTReg JDI Tests failing on Windows #1711
Comments
@smlambert can you take a look? It seems like most are thread & permission failures |
Those tests were moved from extended bucket to sanity on March 31 as jdk_jdi is part of tier1 openjdk tests. #1695 |
Yes, so its likely not a new failure, they used to be in extended.sanity, where they may have been failing for some time now, though I see that I had not enabled weekly runs of extended.openjdk on windows like the other platforms, as we were low on machines (so I can't look at a history). The interesting part is that its tests that are .sh scripts that fail. We should check if only on certain machines, are there permissions that are not allowing the shell scripts to run on all or some our windows machines. |
I've kicked off a few sanity.openjdk grinder runs on several different machines that could be affected.
|
however, there is no need to run the top-level target (which takes a long time), it is sufficient to run the sub-target, jdk_jdi in the Grinder (and thanks for launching some Grinders @M-Davies!) |
I aborted them all just before saw your edit @smlambert 😆. No harm done, I've edited the table above with links to new grinders. Should be a little quicker than from the top level target (thanks for that hint) |
Looks like it's all of them. Possibly a silly question, but do the jenkins users have permission to make their own folders/files? I noticed a lot of the tests are spitting out these sort of errors
|
I believe you have hit upon the root cause of these failures. I know we have had permissions problems on windows very early on in this project, though do not know that these are the same, and not sure specifically what infra actions resolved them, but think this may need to become an infra issue. |
@sxa555 Could you take a look at this? IIRC we've been having perm failures recently with some boxes? Or is that my imagination |
Why would anything be trying to write into the root of the filesystem? The jobs shoudn't have access to write to that location on any system. If it does I'd consider it a bug ... |
That is an odd piece of logic but suggests the |
yes, indeed its not set... not sure what happens on other platforms from this line in that code: |
Exclude PR #1724 It should be possible to reproduce this in a grinder. I could investigate this. See what the test is doing further up |
Ah I see https://github.com/M-Davies/openjdk-jdk8u/blob/baec42c5baec170b8da5a2fae4388104530b6262/jdk/test/com/sun/jdi/ArrayLengthDumpTest.sh#L92-L104 and other tests use |
I ran a grinder with some debug output from https://github.com/M-Davies/openjdk-jdk8u/blob/5715a65e47dac6060cf925c26d408f2bbd5db457/jdk/test/com/sun/jdi/ShellScaffold.sh#L277-L284 to see what the values of some key variables are.
I also see that tmpFile has a forward slash in there when it should really have a backslash to comply with windows file path syntax's. Im not sure if this matters here? @sxa ? |
In your code the blog that sets
Whereas the original code that Shelley pointed at was:
Has something changed? $TEMP should be |
This is a discrepancy between 8 and 9+. This was fixed in https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/3ea9823cb503 (Bug ID https://bugs.openjdk.java.net/browse/JDK-8172188) but was not backported to jdk8 for some reason??? Seeing as it is affecting a large number of tests, this should be probably be contributed upstream @sxa @smlambert do you agree? |
@M-Davies Also those shell scripts have been rewritten to java from jdk12+. https://bugs.openjdk.java.net/browse/JDK-8201652 I've talked with @jerboaa in jbs channel https://adoptopenjdk.slack.com/archives/CF3CGHGHW/p1586294477011100 and believe the reason it's not backported to jdk8 is just help-needed:) https://adoptopenjdk.slack.com/archives/CF3CGHGHW/p1586335628012900 |
Ah ok. Thanks @sophia-guo . I've opened up a request to the team here in Hursley to see if anyone can pick this up as there's quite a lot on my plate at the moment |
Same story for #1718 |
I haven't seen any indication of these tests failing in the last few days. Closing... |
same as #1737. Reopen. |
Describe the bug
We currently have approximately 36 JTReg tests consistently failing on JDK8/hs/x32_windows and JDK8/hs/x64_windows&&x32_windows nightlies. It's the same group every night since the 31st March and they're all
com/sun/jdi/*.sh
tests.FAILING TESTS
The text was updated successfully, but these errors were encountered: