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



src/slave/containerizer/mesos/containerizer.hpp
<https://reviews.apache.org/r/30510/#comment120557>

    "symlinks" seems like an implementation detail.
    
    how about 
    
    s/updateVolumeSymlinks/updateVolumes/



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120568>

    for consistency with how you did it in update() below, do you want to pull 
this down after updateVolumes()?



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120560>

    s/symlinks for//



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120559>

    s/volume symlinks/volumes/



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120562>

    just point to the comments in launch(), so that there is only one place to 
update the comments in the future.



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120563>

    s/resource/resources/



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120565>

    if you agree with my comment in launch(), you dont need to take "current" 
resources as an argument and get it from 'container'?



src/slave/containerizer/mesos/containerizer.cpp
<https://reviews.apache.org/r/30510/#comment120576>

    CHECK(resource.disk().has_persistence()); ?



src/tests/persistent_volume_tests.cpp
<https://reviews.apache.org/r/30510/#comment120578>

    s/according/according to/



src/tests/persistent_volume_tests.cpp
<https://reviews.apache.org/r/30510/#comment120579>

    Do you also want to test for unlinking?


- Vinod Kone


On Feb. 23, 2015, 7:06 a.m., Jie Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30510/
> -----------------------------------------------------------
> 
> (Updated Feb. 23, 2015, 7:06 a.m.)
> 
> 
> Review request for mesos, Ben Mahler, Ian Downes, Michael Park, and Vinod 
> Kone.
> 
> 
> Bugs: MESOS-2031
>     https://issues.apache.org/jira/browse/MESOS-2031
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Allowed Mesos containerizer to prepare and update volumes.
> 
> Mesos containerizer setup volumes for the container.
> 
> 
> Diffs
> -----
> 
>   src/slave/containerizer/mesos/containerizer.hpp 
> 074a2d82dcd882e52f8cd62ed7493295596acb26 
>   src/slave/containerizer/mesos/containerizer.cpp 
> d5b90d12d63becfeb4c3efa9c6f5d826417f582c 
>   src/tests/persistent_volume_tests.cpp 
> b4f2298069233b27f6b4b8d668753810f43bb58d 
> 
> Diff: https://reviews.apache.org/r/30510/diff/
> 
> 
> Testing
> -------
> 
> make check
> 
> 
> Thanks,
> 
> Jie Yu
> 
>

Reply via email to