Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Mesos Reviewbot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187868 --- Patch looks great! Reviews applied: [62929] Passed command:

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187869 --- Ship it! Ship It! - Jie Yu On Oct. 12, 2017, 3:34 p.m.,

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187824 --- FAIL: Some Mesos tests failed. Reviews applied: `['62929']`

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread James DeFelice
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187804 --- Ship it! Ship It! - James DeFelice On Oct. 12, 2017, 3:34

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Benjamin Bannier
> On Oct. 12, 2017, 4:53 p.m., James DeFelice wrote: > > include/mesos/v1/mesos.proto > > Lines 1328 (patched) > > > > > > need the same s/source/volume/ changes for the v1 protos Of course, sorry. Fixed now. -

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/ --- (Updated Oct. 12, 2017, 5:34 p.m.) Review request for mesos, James DeFelice,

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread James DeFelice
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187797 --- include/mesos/v1/mesos.proto Lines 1328 (patched)

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Benjamin Bannier
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/ --- (Updated Oct. 12, 2017, 4:01 p.m.) Review request for mesos, James DeFelice,

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread James DeFelice
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187791 --- include/mesos/mesos.proto Lines 1347 (patched)

Re: Review Request 62929: Added documentation to CSI-related source proto fields.

2017-10-12 Thread Mesos Reviewbot Windows
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62929/#review187789 --- PASS: Mesos patch 62929 was successfully built and tested.