RFR: 8333358: java/io/IO/IO.java test fails intermittently [v5]
Pavel Rappo
prappo at openjdk.org
Thu Jun 20 09:56:45 UTC 2024
> Please review this fix for an intermittent test failure.
>
> On some configurations, the default `expect` timeout of 10 seconds is insufficient. It is increased to 20; it's hard to imagine a configuration for which that new value would still be insufficient, but we'll see.
>
> Aside from that, test-generated diagnostics are improved: the version of the `expect` command and the duration of each test method are recorded. `output.exp` is modified for robustness and clear indication of the timeout condition.
>
> I was reminded, out-of-band, that test timeouts should be scaled with `test.timeout.factor`. I propose to integrate this PR first and then separately update all the affected tests to scale their `expect` timeouts.
Pavel Rappo 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 six additional commits since the last revision:
- Merge branch 'master' into 8333358
- Merge branch 'master' into 8333358
- Add a comment for future maintainers
- Disable timeout in expect scripts
- Change url as suggested in review
- Initial commit
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19627/files
- new: https://git.openjdk.org/jdk/pull/19627/files/aad8b491..ae111829
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19627&range=04
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19627&range=03-04
Stats: 17482 lines in 318 files changed: 13721 ins; 2088 del; 1673 mod
Patch: https://git.openjdk.org/jdk/pull/19627.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19627/head:pull/19627
PR: https://git.openjdk.org/jdk/pull/19627
More information about the core-libs-dev
mailing list