Fix ContainerRuntimeUtil.ContainerRuntime#isPodman check

[zakkak]

Carelessly broke it in #37389

[yrodiere]

LGTM, thanks!

Note I didn't check against my reproducer that this fixed the problem, because I'm currently working on another branch and can't rebuild Quarkus. But I assume you did :)

[yrodiere]

It's odd that our integration tests didn't catch this though 🤔

[zakkak]

It's odd that our integration tests didn't catch this though 🤔

They use docker, not rootless podman :/

But I assume you did :)

Yes I did :)

[quarkus-bot]

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

[johnmanko]

@zakkak This is just for my own curiosity, but does the docker test (.equals) differ from how podman is checked (==)?

[zakkak]

@johnmanko yes, the isDocker check changed to be more "inclusive" and avoid having it look like:

return this == DOCKER || this == DOCKER_ROOTLESS || this == WSL || this == WSL_ROOTLESS;

The reason I didn't want to change isPodman (despite I ended up doing that by mistake) as well was to minimize the delta (in retrospect that didn't go that well though 😅 )