remove unused HostPath from /copy body

[duglin]

Signed-off-by: Doug Davis dug@us.ibm.com

The HostPath json property currently placed in the http body of the /containers/{id}/copy API is unused by the daemon/server so its misleading to include it - just cleaning something up that I noticed.

[vieux]

NOT LGTM, the copy endpoint will be reworked soon. Such a change, even useless, would require an API bump. This is too mush trouble for nothing.

[duglin]

I'm not following. This doesn't require an API bump since the API only calls for the Resource field not the HostPath field to be passed in. IMO the client, right now, is actually violating the API by adding an extra, unused and unneeded, field.

[vieux]

ping @crosbymichael you added HostPath

[duglin]

@vieux, while we're waiting for @crosbymichael to respond, can you elaborate on the changes to 'copy' that are being planned? I was thinking about submitting a PR for option number 3 from my note: https://groups.google.com/forum/#!topic/docker-dev/TmaCiWIEyVk
and would be interesting in knowing the plans before I started.

[vieux]

@duglin copy working both ways i.e. container to host and host to container. See #6580

[duglin]

@crosbymichael any comment on this?

[crosbymichael]

I think for future changes @vieux is right. We should keep it for now

[duglin]

Any idea why it was added to begin with when the server doesn't use/need it?