On Wed, 30 Aug 2023 02:49:09 GMT, David Holmes <dhol...@openjdk.org> 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

Reply via email to