On Fri, 9 Feb 2024 11:38:07 GMT, Darragh Clarke <dcla...@openjdk.org> wrote:
> Currently these tests ignore vm flags, In most cases I've updated them to use > `ProcessTools.createTestJavaProcessBuilder` this usually required some > cleanup also. > > `test/jdk/java/net/ServerSocket/AcceptCauseFileDescriptorLeak.java` and > `test/jdk/java/net/URLConnection/6212146/TestDriver.java` have been set to > use `@require vm.flagless` because they both use `sh` commands. > > I've ran these changes against tiers 1-3 and everything seems stable This pull request has now been integrated. Changeset: 9538f5d3 Author: Darragh Clarke <dcla...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/9538f5d317972bbb82f7f2575819d35d2a5f8b91 Stats: 123 lines in 14 files changed: 27 ins; 35 del; 61 mod 8319651: Several network tests ignore vm flags when start java process Reviewed-by: dfuchs, jpai ------------- PR: https://git.openjdk.org/jdk/pull/17787
