----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66051/#review200013 -----------------------------------------------------------
src/master/http.cpp Lines 1511-1513 (patched) <https://reviews.apache.org/r/66051/#comment280611> "Only agent default resources are supported right now." Also, probably return a `BadRequest` instead of triggering a check failure? Ditto on `shrinkVolume`. src/master/validation.cpp Lines 198 (patched) <https://reviews.apache.org/r/66051/#comment280612> "Only agent default resources are supported" Ditto below. - Chun-Hung Hsiao On March 26, 2018, 4:54 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66051/ > ----------------------------------------------------------- > > (Updated March 26, 2018, 4:54 p.m.) > > > Review request for mesos, Chun-Hung Hsiao, Gaston Kleiman, and Greg Mann. > > > Bugs: MESOS-4965 > https://issues.apache.org/jira/browse/MESOS-4965 > > > Repository: mesos > > > Description > ------- > > Implemented operator API to grow and shrink persistent volume. > > > Diffs > ----- > > src/master/http.cpp f84089e6dcf366716b779f7eff227d1f96d7abda > src/master/master.hpp 7e5d1df1ecb5a0db15782b2480fb7029f8a93ee2 > src/master/master.cpp 390fbe9fe71981101ed2bd9a1f1b21b0e3727b7b > src/master/validation.cpp 9254cd22e67d9aca53135701575fb0857ff6f7d6 > > > Diff: https://reviews.apache.org/r/66051/diff/6/ > > > Testing > ------- > > > Thanks, > > Zhitao Li > >