On Tue, 10 Dec 2024 16:12:55 GMT, Ivan Šipka <isi...@openjdk.org> wrote:
>> @AlanBateman @mahendrachhipa @bwhuang-us @serhiysachkov @mcimadamore >> @JornVernee >> >> adding as manual tests > > Ivan Šipka has updated the pull request incrementally with one additional > commit since the last revision: > > both tests declared automatic, labeled as jdk_foreign_stress and that group > added to tier3 test/jdk/TEST.groups line 96: > 94: :jdk_jpackage \ > 95: :jdk_since_checks \ > 96: :jdk_foreign_stress Shouldn't it be in tier4? test/jdk/TEST.groups line 385: > 383: jdk_foreign = \ > 384: java/foreign \ > 385: jdk/internal/reflect/CallerSensitive/CheckCSMs.java Please exclude the `jdk_foreign_stress` group from the regular `jdk_foreign` test group as well. We regularly run `jdk_foreign` as part of the normal dev cycle. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22580#discussion_r1878706279 PR Review Comment: https://git.openjdk.org/jdk/pull/22580#discussion_r1878706306