RFR: 8294402: Add diagnostic logging to VMProps.checkDockerSupport [v4]

[Mikhailo Seledtsov]

On Wed, 1 Feb 2023 23:18:59 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> If I remove this conditional guard then we will see extra output at the beginning of test run which might be undesirable. For instance: 
>> 
>> jtreg -v1 -conc:4 -Djtreg.log.vmprops=true open/test/hotspot/jtreg/containers/docker/                    
>> 
>> VMProps: Entering call()
>> VMProps: Entering dockerSupport()
>> VMProps: dockerSupport(): platform check: isSupported = false
>> VMProps: dockerSupport(): returning isSupported = false
>> VMProps: Leaving call()
>> Test results: no tests selected
>> 
>> 
>> I am not sure what is the best thing to do here, so I stayed on a "safe" side and added this conditional guard. The model in my mind is that such logging is normally off but can be turned on in case the issue is suspected. However if you believe it should be always on I can remove the guard.
>
> The failure happens so rarely I think you need to leave the option on in order to get
> any useful information about this issue when it occurs in Mach5. If a sighting happens
> in Mach5, I don't think rerunning another task on the same machine will necessarily
> fail so you need to get the information from the failures that do occur.

Thanks Dan and David. I will remove the condition then, and have this logging always on.

-------------

PR: https://git.openjdk.org/jdk/pull/12239