Integrated: 8316693: Simplify at-requires checkDockerSupport()

Thu Oct 12 16:32:35 UTC 2023

On Tue, 10 Oct 2023 00:06:12 GMT, Mikhailo Seledtsov <mseledtsov at openjdk.org> wrote:

> Please review this simple change.
> 
> Currently JTReg @requires extension test/jtreg-ext/requires/VMProps.java checkDockerSupport() uses "<CONTAINER_ENGINE> ps" command to check whether test host/environment allows for container testing. With this change proposing to switch it to "which <CONTAINER_ENGINE>":
>   - the "ps" command could be heavy-weight in some environments, taking longer time then expected
>   - it may not apply to all container engine types

This pull request has now been integrated.

Changeset: 16fd43c5
Author:    Mikhailo Seledtsov <mseledtsov at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/16fd43c5eb0fb18e93f00b5817d368df5afad969
Stats:     4 lines in 1 file changed: 1 ins; 0 del; 3 mod

8316693: Simplify at-requires checkDockerSupport()

Reviewed-by: iklam, lmesnik

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

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