separate healthcheck and container log paths

[baude]

instead of using the container log path to derive where to put the healthchecks, we now put them into the rundir to avoid collision of health check log files when the log path is set by user.

Fixes: #5915

Signed-off-by: Brent Baude [email protected]

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: baude

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [baude]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[mheon]

LGTM

[rhatdan]

/lgtm

[rhatdan]

Should this be back ported to podman v1.9?

[cjeanner]

Oh, sad to see it moving back in a location hard to share across containers. Anyway, as long as it solves the issue, it's good for me :).
I think it should be backported to 1.9 as well.

[EmilienM]

Should this be back ported to podman v1.9?

Yes, please. We'll need it backported into RHEL 8.x if possible so our healthchecks are correctly logged again.

Thanks

[baude]

guys were are way ahead of you ... see #6008

[EmilienM]

guys were are way ahead of you ... see #6008

Thanks ;-)