[CRIU] [PATCH p.haul] Pin docker C/R version

[Pavel Emelyanov]

On 11/03/2015 01:15 AM, Hui Kang wrote:
> The latest Docker-cr uses a different location for docker daemon (,
> i.e., /var/lib/docker/0.0/) than the hardcoded one. Luckily I had an
> old version of docker-cr in my git repo.
> 
> So we can either use the old version of docker (as this patch) or
> change the location in p.haul docker driver.
> 
> @Pavel, which one do you think is a better choice?

Until we have C/R support in Docker, I think any choice will go :) So
for now it's up to you.

> - Hui
> 
> On Mon, Nov 2, 2015 at 5:11 PM, Marcos Lilljedahl <marcosnils at gmail.com> wrote:
>> Hi Hui,
>>
>> May I ask why you changed the default c/r fork in replace for yours?. What
>> did you add that the original one doesn't have?
>>
>> Marcos.
>>
>> On Sun, Nov 1, 2015 at 5:44 PM, Hui Kang <hkang.sunysb at gmail.com> wrote:
>>>
>>> The latest version changes the container filesystem location
>>>
>>> Signed-off-by: Hui Kang <hkang.sunysb at gmail.com>
>>> ---
>>>  test/docker/HOWTO | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/test/docker/HOWTO b/test/docker/HOWTO
>>> index 3255360..6a5ef8d 100644
>>> --- a/test/docker/HOWTO
>>> +++ b/test/docker/HOWTO
>>> @@ -13,9 +13,9 @@ your running containers.
>>>
>>>     The docker version that supports checkpoint and restore can be
>>> obtained by
>>>
>>> -   # git clone https://github.com/boucher/docker.git
>>> +   # git clone https://github.com/huikang/docker.git
>>>     # cd docker.git
>>> -   # git checkout cr-combined
>>> +   # git checkout cr-combined-1006
>>>
>>>     On both nodes, compile and store the the docker binary as
>>>
>>> --
>>> 1.9.1
>>>
>>> _______________________________________________
>>> CRIU mailing list
>>> CRIU at openvz.org
>>> https://lists.openvz.org/mailman/listinfo/criu
>>
>>
> .
>