mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chun-Hung Hsiao <chhs...@mesosphere.io>
Subject Re: Review Request 63021: Added `getService()` function to launch CSI plugins.
Date Wed, 15 Nov 2017 02:40:50 GMT

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

(Updated Nov. 15, 2017, 2:40 a.m.)


Review request for mesos, James DeFelice, Jie Yu, Joseph Wu, and Jan Schlicht.


Changes
-------

Used the new `ContainerDaemon` and split some unrelated changes into r63823.


Summary (updated)
-----------------

Added `getService()` function to launch CSI plugins.


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


Repository: mesos


Description (updated)
-------

The `getService()` method first checks if there is already a container
daemon for the specified plugin, and creates a new one if not. The
post-start hook for the container daemon will call `connect()` to
wait for the endpoint socket file to appear and connect to it, then
set up the corresponding promise of CSI client. The post-stop hook will
remove the socket file and create a new promise for the next call to the
post-start hook to set it up.


Diffs (updated)
-----

  src/resource_provider/storage/provider.cpp 46224997430ac0c568904d80014166a6f059907f 


Diff: https://reviews.apache.org/r/63021/diff/8/

Changes: https://reviews.apache.org/r/63021/diff/7-8/


Testing
-------


Thanks,

Chun-Hung Hsiao


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