On 7/16/2026 3:25 AM, Daniel P. Berrangé wrote: > The '--engine' arg accepts either 'podman' or 'docker', which is > not sufficiently granular to map directly to a command. This > means that docker.py still has to then probe the exact command > to use. > > Meanwhile the 'probe' command prints out the full command to use > but this cannot be passed back to docker.py to avoid probing > again, so the caching is only useful in the few case where we > run a container directly bypassing docker.py. > > Address this by replacing --engine with --command for docker.py. > > This in turn requires the --container-engine configure arg to be > replaced with --container-command. > > With these changes the container command is probed at most once > during configure and never again, while running in an unconfigured > tree will still probe on demand. > > Signed-off-by: Daniel P. Berrangé <[email protected]> > --- > > In v3: > > - Avoid bashism when testing probe result > > configure | 25 +++++++------ > tests/docker/Makefile.include | 13 ++++--- > tests/docker/docker.py | 67 ++++++++++++----------------------- > 3 files changed, 40 insertions(+), 65 deletions(-) >
Thanks for the update, it's working fine now. Also, docker.py run command is now fast again when given --command argument. Tested-by: Pierrick Bouvier <[email protected]> Reviewed-by: Pierrick Bouvier <[email protected]> I'll pull this today. Regards, Pierrick
