----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67779/#review205966 -----------------------------------------------------------
include/mesos/mesos.proto Lines 1927-1929 (original), 1927-1932 (patched) <https://reviews.apache.org/r/67779/#comment288903> We don't seem to document removed proto definitions, maybe we should just drop this comment here and below. include/mesos/mesos.proto Lines 1991 (patched) <https://reviews.apache.org/r/67779/#comment288904> Use backticks around `RAW`, `MOUNT`, and `BLOCK`; here and below. include/mesos/mesos.proto Lines 2006 (patched) <https://reviews.apache.org/r/67779/#comment288906> I agree with Jan that `target` is confusing as we don't want users to pass the target resource (i.e., an already "destroy disk resource") here. Calling this something else would make clear that we want the source resource of the operation passed in, not the expect result (contrary to how e.g., a reserve operation is structured). Maybe even go with `source` like in the `Create` op? - Benjamin Bannier On July 10, 2018, 5:10 a.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67779/ > ----------------------------------------------------------- > > (Updated July 10, 2018, 5:10 a.m.) > > > Review request for mesos, Benjamin Bannier, Gastón Kleiman, Greg Mann, James > DeFelice, and Jie Yu. > > > Bugs: MESOS-9066 > https://issues.apache.org/jira/browse/MESOS-9066 > > > Repository: mesos > > > Description > ------- > > This patch introduces `CREATE_DISK` and `DESTROY_DISK` operations for > conversion between RAW disk resaurces and MOUNT/BLOCK disk resources to > replace `CREATE_VOLUME`/`DESTROY_VOLUME`/`CREATE_BLOCK`/`DESTROY_BLOCK`. > > This replacement involves two aspects: > > 1. Renaming: The term "volume" is overloaded and created name conflicts > in mesos. This renaming would lead to cleaner names in ACL and other > places, and is more consistent with CSI. > > 2. Removing PATH support. PATH disks are used to split a disk resource > into smaller chunks for concurrent independent uses. However, this > concept does not quite fit CSI well. > > > Diffs > ----- > > include/mesos/mesos.proto 463e6adcdf14115b2f21270bb1fd9c45e1b67cc3 > include/mesos/v1/mesos.proto 8eaad9c4b2a9cdd527922a004b0f7dd0dc58a7f1 > > > Diff: https://reviews.apache.org/r/67779/diff/2/ > > > Testing > ------- > > Test done later in chain. > > > Thanks, > > Chun-Hung Hsiao > >