Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-10 Thread Lily Chen
n busybox' 1300 times to recreate failure, failing two docker tests. Also ran fix after clearing 'docker ps -a' log and with 300 instances in 'docker ps -a' log to make sure original functionality not broken. Thanks, Lily Chen

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-10 Thread Lily Chen
to make sure original functionality not broken. Thanks, Lily Chen

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-10 Thread Lily Chen
ps -a' log and with 300 instances in 'docker ps -a' log to make sure original functionality not broken. Thanks, Lily Chen

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-11 Thread Lily Chen
0 Thanks, Lily Chen

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-16 Thread Lily Chen
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35330/#review87802 --- On June 11, 2015, 9:39 p.m., Lily Chen wrote: > > ---

Re: Review Request 35330: Capped number of parallel inspect instances on a docker ps call.

2015-06-16 Thread Lily Chen
ly just return the > list of all the results, so I think we can then do a collect call on all the > futures to aggregate the results. > > Lily how does that sound to you? This is a definitely a good idea. I'll take a look. - Lily ------- This is an automatically generate

Re: Review Request 34136: Add ContainerImage protobuf.

2015-07-01 Thread Lily Chen
org/r/34136/#comment143158> Same issue as when MesosInfo is not present. What happens when there is no Appc message present in Image? - Lily Chen On June 22, 2015, 4:42 p.m., Ian Downes wrote: > > --- > This is an automati

Re: Review Request 34142: AppC provisioner.

2015-07-01 Thread Lily Chen
lure message to something like "Provisioner and Image type mismatch" src/slave/containerizer/provisioners/appc.cpp (lines 484 - 487) <https://reviews.apache.org/r/34142/#comment143174> Do we want to remove everything on a failure? This could make finding the failure point d

Review Request 37196: Add Docker Image type to Container Image protobuf.

2015-08-06 Thread Lily Chen
, Lily Chen

Review Request 37197: Docker image store.

2015-08-06 Thread Lily Chen
/provisioners/docker/store.cpp PRE-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-06 Thread Lily Chen
/37198/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37199: Added store interface and moved store implementation to LocalStore subclass.

2015-08-06 Thread Lily Chen
-CREATION Diff: https://reviews.apache.org/r/37199/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-06 Thread Lily Chen
-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-07 Thread Lily Chen
ality as with appc. How might we consolidate? With a common provision function? - Lily --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37198/#review94475 ----

Re: Review Request 37197: Docker image store.

2015-08-07 Thread Lily Chen
) <https://reviews.apache.org/r/37197/#comment149188> Layers no longer in store. src/slave/containerizer/provisioners/docker/store.hpp (line 128) <https://reviews.apache.org/r/37197/#comment149189> fetcher is actually never used, maybe I should just get rid of it - Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-07 Thread Lily Chen
sit: https://reviews.apache.org/r/37197/#review94448 --- On Aug. 6, 2015, 10:51 p.m., Lily Chen wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 37196: Add Docker Image type to Container Image protobuf.

2015-08-07 Thread Lily Chen
Diff: https://reviews.apache.org/r/37196/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-07 Thread Lily Chen
/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-07 Thread Lily Chen
-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37199: Added store interface and moved store implementation to LocalStore subclass.

2015-08-07 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37199/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-07 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37245: Refactor Docker Image to exclude path and manifest.

2015-08-07 Thread Lily Chen
/provisioners/docker/backend.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37245/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37246: Refactor store to use updated DockerImage.

2015-08-07 Thread Lily Chen
://reviews.apache.org/r/37246/diff/ Testing --- make check Thanks, Lily Chen

Review Request 37247: Added Docker image reference store.

2015-08-07 Thread Lily Chen
/docker/reference_store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37196: Add Docker Image type to Container Image protobuf.

2015-08-10 Thread Lily Chen
Diff: https://reviews.apache.org/r/37196/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-10 Thread Lily Chen
/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-10 Thread Lily Chen
-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-10 Thread Lily Chen
c213ac779e7acc3235312ca9524b3959417b8c33 src/slave/containerizer/provisioners/docker/reference_store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-10 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION src/slave/flags.hpp 881d494c06fea5c382d27b357d65c1baf201ae46 src/slave/flags.cpp 82b6cf47af26f0533ff603a67240777e9a9b986e Diff: https://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-11 Thread Lily Chen
, visit: https://reviews.apache.org/r/37247/#review94646 --- On Aug. 10, 2015, 10:53 p.m., Lily Chen wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37247/ > --

Re: Review Request 37197: Docker image store.

2015-08-11 Thread Lily Chen
/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-11 Thread Lily Chen
Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37199: Added store interface and moved store implementation to LocalStore subclass.

2015-08-11 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37199/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-11 Thread Lily Chen
tps://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37245: Refactor Docker Image to exclude path and manifest.

2015-08-11 Thread Lily Chen
/docker.cpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37245/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37246: Refactor store to use updated DockerImage.

2015-08-11 Thread Lily Chen
/provisioners/docker/store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37246/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-11 Thread Lily Chen
/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37382: Introduced provisioner Backend interface.

2015-08-12 Thread Lily Chen
tps://reviews.apache.org/r/37382/#comment149928> Should this return a Future since the provisioner interface returns a Future for destroy? - Lily Chen On Aug. 12, 2015, 5:49 a.m., Jiang Yan Xu wrote: > > --- > This is a

Re: Review Request 37427: Docker registry: adding TokenManager.

2015-08-12 Thread Lily Chen
https://reviews.apache.org/r/37427/#comment150089> DockerContainerizerTest tests the Docker Containerizer. The TokenManager is a component of the Mesos Containerizer, and would therefore be a part of a Docker provisioner test file. - Lily Che

Re: Review Request 37197: Docker image store.

2015-08-14 Thread Lily Chen
://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37199: Added store interface and moved store implementation to LocalStore subclass.

2015-08-14 Thread Lily Chen
/docker/store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37199/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-14 Thread Lily Chen
/slave/flags.cpp e43dd1c13dd4263dc326842233808ddb7a9bb74c src/tests/containerizer/provisioner.hpp c4ba46794fe5d7875fda11155367f521c34ea339 Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-14 Thread Lily Chen
Thanks, Lily Chen

Re: Review Request 37245: Refactor Docker Image to exclude path and manifest.

2015-08-14 Thread Lily Chen
/docker.cpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37245/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37246: Refactor store to use updated DockerImage.

2015-08-14 Thread Lily Chen
/provisioners/docker/store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37246/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-14 Thread Lily Chen
later review. Thanks, Lily Chen

Review Request 37495: Docker provisioner local store unit tests.

2015-08-14 Thread Lily Chen
="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-14 Thread Lily Chen
/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37496/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-14 Thread Lily Chen
--gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-14 Thread Lily Chen
="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-14 Thread Lily Chen
-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-14 Thread Lily Chen
/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37497/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-16 Thread Lily Chen
/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-16 Thread Lily Chen
e43dd1c13dd4263dc326842233808ddb7a9bb74c src/tests/containerizer/provisioner.hpp c4ba46794fe5d7875fda11155367f521c34ea339 Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37199: Added store interface and moved store implementation to LocalStore subclass.

2015-08-16 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37199/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-16 Thread Lily Chen
/containerizer/provisioners/docker/store.cpp PRE-CREATION src/slave/flags.hpp e56738e2dfd6593ef8f093687919da287af78f77 src/slave/flags.cpp e43dd1c13dd4263dc326842233808ddb7a9bb74c Diff: https://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37245: Refactor Docker Image to exclude path and manifest.

2015-08-16 Thread Lily Chen
/docker.cpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/backend.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37245/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37246: Refactor store to use updated DockerImage.

2015-08-16 Thread Lily Chen
/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37246/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-16 Thread Lily Chen
-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-16 Thread Lily Chen
PRE-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-16 Thread Lily Chen
="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-16 Thread Lily Chen
-CREATION src/slave/containerizer/provisioners/docker/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37497/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-19 Thread Lily Chen
/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-19 Thread Lily Chen
e56738e2dfd6593ef8f093687919da287af78f77 src/slave/flags.cpp b36710d6d7a7250bc071a57310a2d54bfb3bc624 src/tests/containerizer/provisioner.hpp c4ba46794fe5d7875fda11155367f521c34ea339 Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-19 Thread Lily Chen
://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-19 Thread Lily Chen
/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-19 Thread Lily Chen
/tests/containerizer/docker_provisioner_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-19 Thread Lily Chen
="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-19 Thread Lily Chen
-CREATION src/slave/containerizer/provisioners/docker/reference_store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37497/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-24 Thread Lily Chen
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37197/#review95830 --- On Aug. 19, 2015, 6:41 p.m., Lily Chen wrote: > > --- > This is

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-24 Thread Lily Chen
Name struct too. If there are more than 2 path components, what would constitute as the registry name? Simply the first path component? - Lily --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37198/#review95831 ---

Re: Review Request 37197: Docker image store.

2015-08-24 Thread Lily Chen
Diff: https://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-24 Thread Lily Chen
, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-24 Thread Lily Chen
/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-24 Thread Lily Chen
/tests/containerizer/docker_provisioner_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-24 Thread Lily Chen
="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-24 Thread Lily Chen
est_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-25 Thread Lily Chen
://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-25 Thread Lily Chen
://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-25 Thread Lily Chen
e56738e2dfd6593ef8f093687919da287af78f77 src/slave/flags.cpp b36710d6d7a7250bc071a57310a2d54bfb3bc624 src/tests/containerizer/provisioner.hpp c4ba46794fe5d7875fda11155367f521c34ea339 Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-25 Thread Lily Chen
://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-25 Thread Lily Chen
-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-25 Thread Lily Chen
9fd71d1ddf442712977596e7a13969ff5c1d68db src/tests/containerizer/docker_provisioner_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-25 Thread Lily Chen
./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-25 Thread Lily Chen
_shuffle=1 Thanks, Lily Chen

Re: Review Request 37197: Docker image store.

2015-08-25 Thread Lily Chen
://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-25 Thread Lily Chen
e56738e2dfd6593ef8f093687919da287af78f77 src/slave/flags.cpp b36710d6d7a7250bc071a57310a2d54bfb3bc624 src/tests/containerizer/provisioner.hpp c4ba46794fe5d7875fda11155367f521c34ea339 Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-25 Thread Lily Chen
://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-25 Thread Lily Chen
-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37495: Docker provisioner local store unit tests.

2015-08-25 Thread Lily Chen
571e1ac0f96b2452797a478680b540f2aab63aab src/tests/containerizer/docker_provisioner_tests.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37495/diff/ Testing --- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-25 Thread Lily Chen
./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

Re: Review Request 37497: Added Docker provisioner paths which handles path manipulation.

2015-08-25 Thread Lily Chen
_shuffle=1 Thanks, Lily Chen

Re: Review Request 37427: Docker registry: adding TokenManager.

2015-08-25 Thread Lily Chen
7/#comment151791> Fix implementatio typo and period at the end of comment. src/tests/provisioners/docker_provisioner_tests.cpp (line 562) <https://reviews.apache.org/r/37427/#comment151799> extra space here. - Lily Chen On Aug. 24, 2015, 5:16 p.m., Jojy Varghese wrote: >

Re: Review Request 37197: Docker image store.

2015-08-26 Thread Lily Chen
ename is no longer necessary when the DockerImage scheme changes in 37200. - Lily --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37197/#review96496 ------

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-26 Thread Lily Chen
ction need to be captured in this lambda. - Lily --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37200/#review96561 --- On Aug. 26,

Re: Review Request 37197: Docker image store.

2015-08-27 Thread Lily Chen
b36710d6d7a7250bc071a57310a2d54bfb3bc624 Diff: https://reviews.apache.org/r/37197/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37198: Add Docker image provisioner and copy backend.

2015-08-27 Thread Lily Chen
b36710d6d7a7250bc071a57310a2d54bfb3bc624 src/tests/containerizer/provisioner.hpp a26b8138d8cc3086058b15a797dd15354a84019f Diff: https://reviews.apache.org/r/37198/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37200: Refactored DockerImage struct to store a list of layer ids instead of linked list of DockerLayers.

2015-08-27 Thread Lily Chen
PRE-CREATION src/slave/flags.hpp e56738e2dfd6593ef8f093687919da287af78f77 src/slave/flags.cpp b36710d6d7a7250bc071a57310a2d54bfb3bc624 Diff: https://reviews.apache.org/r/37200/diff/ Testing --- make check Thanks, Lily Chen

Re: Review Request 37247: Added Docker image reference store.

2015-08-27 Thread Lily Chen
/docker/store.hpp PRE-CREATION src/slave/containerizer/provisioners/docker/store.cpp PRE-CREATION Diff: https://reviews.apache.org/r/37247/diff/ Testing --- make check Tests will be added in a later review. Thanks, Lily Chen

Re: Review Request 37496: Move docker provisioner local store into dedicated folders.

2015-08-27 Thread Lily Chen
--- sudo make check ./bin/mesos-tests.sh --gtest_filter="*DockerProvisioner*" --gtest_repeat=20 --gtest_shuffle=1 Thanks, Lily Chen

  1   2   >