add config option for ChownCopiedFiles

[matejvasek]

A new container engine config boolean determining whether files copied into a container should have changed ownership to the primary uid/gid of the container.

[matejvasek]

/cc @vrothberg @rhatdan

[rhatdan]

LGTM
Podman currently does this by default?
/approve

[matejvasek]

@rhatdan it's little bit more complicated
the podman CLI does chown by default (so it doesn't break existing user code),
however the Docker APIv2 compat endpoints does not do chown (so it doesn't break docker api client expectations)

[matejvasek]

@rhatdan I think that the option should only affect the CLI not the compat endpoint

[rhatdan]

SGTM
@containers/podman-maintainers PTAL

[vrothberg]

LGTM, thank you, @matejvasek

[vrothberg]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matejvasek, rhatdan, vrothberg

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 [rhatdan,vrothberg]

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