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 functions to launch CSI plugin in storage local resource provider.
Date Sat, 04 Nov 2017 01:06:23 GMT

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

(Updated Nov. 4, 2017, 1:06 a.m.)


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


Changes
-------

Updated using the new `storage::paths` helper functions.


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


Repository: mesos


Description (updated)
-------

During initialization, the storage local resource provider first tries
to recover its ID of the last session through reading the actual path
linked by `<work_dir>/meta/resource_providers/<type>/<name>/latest`,
then subscribe to the agent's resource provider manager. If this is a
new subscription, it will set up a new dir for CSI plugins to put their
socket files.

Once the CSI socket dir is set up, the storage local resource provider
can connect to a CSI plugin through the following procedure:
1. It first tries to connect to the existing socket file if there is
   one. On success, just return the connection.
2. On failure, kill the running plugin and remove the socket file.
3. Launch a container to run the plugin, and w for the socket file to
   appear, then connect to the socket file.


Diffs (updated)
-----

  src/resource_provider/storage/provider.cpp 46224997430ac0c568904d80014166a6f059907f 


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

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


Testing
-------


Thanks,

Chun-Hung Hsiao


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