mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jie Yu <yujie....@gmail.com>
Subject Re: Review Request 64868: Added initial doc about CSI support in Mesos.
Date Wed, 03 Jan 2018 00:51:06 GMT


> On Dec. 29, 2017, 3:03 p.m., James DeFelice wrote:
> > docs/csi.md
> > Lines 151 (patched)
> > <https://reviews.apache.org/r/64868/diff/2/?file=1929134#file1929134line151>
> >
> >     Can we make it clear that we're talking about a hypothetical EBS plugin? AFAIK
the external resource provider stuff isn't in place yet (and may contain unexpected monsters)

I used LVM instead.


> On Dec. 29, 2017, 3:03 p.m., James DeFelice wrote:
> > docs/csi.md
> > Lines 369 (patched)
> > <https://reviews.apache.org/r/64868/diff/2/?file=1929134#file1929134line369>
> >
> >     it's worth noting that because this is based on the CSI specification, there's
an implicit dependency between backward compatibility of the module interface and the CSI
specification version. In other words, there's already a compatiblity scheme for Mesos modules
that was crafted prior to CSI. CSI doesn't provide a backward compatibility promise. As CSI
evolves, this module interface may implicitly evolve in a way that deviates from the compatibility
promise described here: https://github.com/apache/mesos/blob/master/docs/modules.md#module-versioning-and-backwards-compatibility
> >     
> >     .. or, maybe I misunderstand our guarantees re: module backwards compat.

Clarified, module has to be rebuilt against each release of Mesos.


- Jie


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


On Dec. 29, 2017, 4:53 a.m., Jie Yu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64868/
> -----------------------------------------------------------
> 
> (Updated Dec. 29, 2017, 4:53 a.m.)
> 
> 
> Review request for mesos, Benjamin Bannier, Chun-Hung Hsiao, Gaston Kleiman, Greg Mann,
Joseph Wu, and Jan Schlicht.
> 
> 
> Repository: mesos
> 
> 
> Description
> -------
> 
> Added initial doc about CSI support in Mesos.
> 
> 
> Diffs
> -----
> 
>   docs/csi.md PRE-CREATION 
> 
> 
> Diff: https://reviews.apache.org/r/64868/diff/2/
> 
> 
> Testing
> -------
> 
> The rendering can be checked here:
> https://github.com/jieyu/mesos/blob/csi_doc/docs/csi.md
> 
> 
> Thanks,
> 
> Jie Yu
> 
>


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