Allow copying files/dirs to a container - simpler implementation#13125
Allow copying files/dirs to a container - simpler implementation#13125cpuguy83 wants to merge 1 commit intomoby:masterfrom
Conversation
api/client/cp.go
Outdated
There was a problem hiding this comment.
s/and/for/ on this line and next.
api/client/cp.go
Outdated
There was a problem hiding this comment.
Until we support container <-> container stuff it might be better to write this as something like:
[CONTAINER:PATH LOCALPATH|-] | [LOCALPATH|- CONTAINER:PATH]
|
ping @jlhawn |
cpuguy83
force-pushed
the
simpler_copy_in
branch
3 times, most recently
from
4fa34a7 to
5e52b5e
Compare
|
Windows failure is due to some parsing issues in determining if the arg is a file path or a container+path. Will have to look more at it later. But otherwise if we agree we want this functionality (and I think we have in the other PR) I'd like to get this moved to code-review. |
cpuguy83
force-pushed
the
simpler_copy_in
branch
2 times, most recently
from
c32252e to
2444f20
Compare
Signed-off-by: Brian Goff <[email protected]>
|
Windows path errors all fixed. |
|
Lets not do competing implementations of exiting PRs. Lets start working with the original authors and contributors can collaborate on their forks and get the implementations worked out and have a joint PR opened that everyone is proud of. |
5 participants
This is a simpler implementation of #10198
Does not support copying files between containers, though this could be done via:
When copying from a container it will use
GET /containers/<id>/copy, copying to a container would bePOST /containers/<id>/copy?to=<path>The
postContainersCopymethod will proxy togetContainersCopyfor older API versions.Closes #10198 #5846