----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63021/ -----------------------------------------------------------
(Updated Oct. 17, 2017, 12:40 a.m.) Review request for mesos, James DeFelice, Jie Yu, Joseph Wu, and Jan Schlicht. Changes ------- Refactored. Summary (updated) ----------------- Added functions to launch CSI plugin in storage local resource providers. 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 and CSI socket dir of the last session through reading the actual path linked by `work_dir/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/2/ Changes: https://reviews.apache.org/r/63021/diff/1-2/ Testing ------- Thanks, Chun-Hung Hsiao