mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ilya Pronin <ipro...@twopensource.com>
Subject Re: Review Request 56174: Added skipping already stored layers to local Docker puller.
Date Tue, 07 Feb 2017 21:10:28 GMT


> On Feb. 5, 2017, 1:28 a.m., Jie Yu wrote:
> > src/tests/containerizer/provisioner_docker_tests.cpp, lines 262-270
> > <https://reviews.apache.org/r/56174/diff/2/?file=1623553#file1623553line262>
> >
> >     Instead of testing local puller directly, I'd suggest we test using 'Store'.
This test assumes too many impl details which will be very fragile as code evolves. We try
to avoid that if possible.
> >     
> >     My suggestion is that we pull the image once, making sure store has it. And
then, delete one layer, and then pull again. Make sure the second pull is successful.

Done in [56284](https://reviews.apache.org/r/56284/).


- Ilya


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56174/#review164257
-----------------------------------------------------------


On Feb. 3, 2017, 6:07 p.m., Ilya Pronin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/56174/
> -----------------------------------------------------------
> 
> (Updated Feb. 3, 2017, 6:07 p.m.)
> 
> 
> Review request for mesos, Jie Yu and Timothy Chen.
> 
> 
> Bugs: MESOS-7045
>     https://issues.apache.org/jira/browse/MESOS-7045
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> If a layer is already in the store, there's no need to extract it. `RegistryPuller` already
does this and `Store` is ready for such puller behaviour.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/provisioner/docker/local_puller.cpp ee391af898886bff9e5b911697f725c5ea53ebd8

>   src/tests/containerizer/provisioner_docker_tests.cpp af9987f88205d00d091f35fa734d5667506aaffd

> 
> Diff: https://reviews.apache.org/r/56174/diff/
> 
> 
> Testing
> -------
> 
> Added a test verifying that local puller will skip layes that are already in the store.
Ran `make check`.
> 
> 
> Thanks,
> 
> Ilya Pronin
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message