On Wed, 30 Aug 2023 02:49:09 GMT, David Holmes <[email protected]> wrote:
>> Container ecosystem is growing. It would be beneficial to define custom
>> command to figure out whether a specific test host or environment allows for
>> container testing. This enhancement seeks to make the command used by jtreg
>> "requires" extension configurable, specifically
>> test/jtreg-ext/requires/VMProps.java checkContainerSupport().
>
> test/jtreg-ext/requires/VMProps.java line 538:
>
>> 536: log("Entering containerSupport()");
>> 537:
>> 538: boolean isSupported = true;
>
> This needs to be initialized to false.
Good catch, thank you. This is a bug, a leftover from my experimentation.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15475#discussion_r1309571210
