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

Reply via email to