mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jie Yu <yujie....@gmail.com>
Subject Review Request 65049: Fixed the persistent volume permission issue in DockerContainerizer.
Date Tue, 09 Jan 2018 19:31:42 GMT

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

Review request for mesos, Gilbert Song, James Peach, and Jiang Yan Xu.


Bugs: MESOS-8356
    https://issues.apache.org/jira/browse/MESOS-8356


Repository: mesos


Description
-------

This patch fixes MESOS-8356 by skipping the current container to be
launched when doing the shared volume check (`isVolumeInUse`). Prior to
this patch, the code is buggy because `isVolumeInUse` will always be set
to `true`.


Diffs
-----

  src/slave/containerizer/docker.cpp 890cb2aba44fe76e891117833eac8ccca00b759b 


Diff: https://reviews.apache.org/r/65049/diff/1/


Testing
-------

sudo make check


Thanks,

Jie Yu


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