RFR: 8362087: Test containers/docker/ShareTmpDir.java intermittent fails [v5]
SendaoYan
syan at openjdk.org
Tue Oct 28 04:22:20 UTC 2025
> Hi all,
>
> The test containers/docker/ShareTmpDir.java intermittent fails, because before this PR, test assume start two java process in docker container by two threads will make the two java process start simultancely, but in fact, the second java process maybe start slower than the first one, even that the first java process already exit and then the second java process not start yet. If we add `Thread.sleep(2000)` to the second thread at the begin of run() method, will make this intermittent failure to always reproduceable. This prove the anasyis.
>
> If the two java process can not start simultancely, the expected '/tmp/hsperfdata_1 locked' error can not observed. So this test will intermittent fails.
>
> This PR will check all the two java processes started already and runing simultancely before exit, this will make the expected '/tmp/hsperfdata_1 locked' error can be alway observed.
>
> The touch file test/hotspot/jtreg/containers/docker/WaitForFlagFile.java only use for test containers/docker/ShareTmpDir.java.
>
> Change has been verified locally, test-fix only, no risk.
SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
- Merge branch 'openjdk:master' into jbs8362087
- Merge branch 'openjdk:master' into jbs8362087
- Merge branch 'openjdk:master' into jbs8362087
- Add synchronized lock for addClassOptions
- 8362087: Test containers/docker/ShareTmpDir.java intermittent fails
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26605/files
- new: https://git.openjdk.org/jdk/pull/26605/files/d04cc981..997451c5
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26605&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26605&range=03-04
Stats: 40778 lines in 955 files changed: 23790 ins; 11903 del; 5085 mod
Patch: https://git.openjdk.org/jdk/pull/26605.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/26605/head:pull/26605
PR: https://git.openjdk.org/jdk/pull/26605
More information about the hotspot-runtime-dev
mailing list