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 63387: Added publish/unpublish in storage local resource provider.
Date Fri, 01 Dec 2017 01:55:16 GMT

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

(Updated Dec. 1, 2017, 1:55 a.m.)


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


Changes
-------

Added `VolumeState` protobuf.


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


Repository: mesos


Description (updated)
-------

Storage local resource provider can now handle PUBLISH events.
Although we don't do UNPUBLISH for now, the unpublish function is
still required for deleting volumes.

This patch also adds a protobuf to keep track of CSI volume states,
which persist across agents.


Diffs (updated)
-----

  src/Makefile.am 4a3b728dc2cbfaf4b10068562778a87a6b331238 
  src/csi/state.hpp PRE-CREATION 
  src/csi/state.proto PRE-CREATION 
  src/csi/utils.hpp 8ee97b2192d00f7e266c297de49315b5ce377308 
  src/csi/utils.cpp 4dd21fbae2070606c36c94415957ea3213a7dad1 
  src/resource_provider/storage/provider.cpp f586afc256fbcb2f2bf2451ffb7e7ba1d59cfa78 


Diff: https://reviews.apache.org/r/63387/diff/4/

Changes: https://reviews.apache.org/r/63387/diff/3-4/


Testing
-------

make


Thanks,

Chun-Hung Hsiao


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