mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 67779: Introduced `CREATE_DISK` and `DESTROY_DISK` operations.
Date Wed, 11 Jul 2018 12:47:00 GMT

-----------------------------------------------------------
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
> 
>


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