support container-to-container copy

[daiaji]

podman cp v2ray:/etc/ssl/certs/ca-certificates.crt caddy:/etc/ssl/certs/ca-certificates.crt
Error: invalid arguments v2ray:/etc/ssl/certs/ca-certificates.crt, caddy:/etc/ssl/certs/ca-certificates.crt you must use just one container

There seems to be no harm in implementing this function.

[zhangguanzhang]

which version did you used to? or , did you tried with the latest version?

[mheon]

Should be technically possible. I'll label this as a feature request.

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

@vrothberg Is working on podman-remote cp now, so maybe he can look at this?

[vrothberg]

@vrothberg Is working on podman-remote cp now, so maybe he can look at this?

Good timing. I am currently rewiring the entire copy code. I will keep this in mind so that we can easily add support for container-to-container copy.

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

@vrothberg Did you make this work?

[vrothberg]

@vrothberg Did you make this work?

No. I focused on host/ctr copying only. The backend code, however, would allow that now, so that (nearly) all code changes would happen cmd/podman.

[rhatdan]

Since this is a feature, we can put it off for a while until we stabilize 3.0

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

This would still be a good feature. I was recently asked about this in buildah copy as well, but in that case they wanted to copy from an image to a container.

[vrothberg]

Agreed, it would be really nice to get this done. I have too many things on my plate at the moment but I'd be more than happy to help others get it done. A word of caution: I do not think it's a good first issue. It will require very extensive testing.

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[infiniteregrets]

hey @vrothberg, are you working on this? If not then, can I try working on it

[vrothberg]

hey @vrothberg, are you working on this? If not then, can I try working on it

Thanks, it's yours :)

As mentioned above, this feature will need extensive testing in https://github.com/containers/podman/blob/master/test/system/065-cp.bats but we can assist. Code wise, the plumbing would happen cmd/podman/containers/cp.go.

[infiniteregrets]

I will try to work on it as soon as possible. I will ping you if I am stuck! Thank you

[github-actions]

A friendly reminder that this issue had no activity for 30 days.

[rhatdan]

@infiniteregrets @vrothberg Did this ever move forward?

[infiniteregrets]

@rhatdan I did move it forward, but I couldnt figure out how to rename a file which does not exist in the destination when copied, so @vrothberg will take over. sorry ):

[vrothberg]

I plan to tackle it this week 👍