----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55954/#review163704 -----------------------------------------------------------
Ship it! Ship It! - Vinod Kone On Jan. 28, 2017, 11:34 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55954/ > ----------------------------------------------------------- > > (Updated Jan. 28, 2017, 11:34 p.m.) > > > Review request for mesos, Jan Schlicht and Vinod Kone. > > > Bugs: MESOS-6991 > https://issues.apache.org/jira/browse/MESOS-6991 > > > Repository: mesos > > > Description > ------- > > To prepare for future work which will enable the > modular fetching of secrets, we should change the > Environment.Variable.Value field from required to > optional. This way, the field can be left empty > and filled in by a secret fetching module. > > > Diffs > ----- > > include/mesos/mesos.proto a08b8fb2edf26ae730467557ae0e33e3d4252593 > include/mesos/v1/mesos.proto 4a4609e7db659252ef7f1ca61f8ba079b901b7d0 > src/checks/checker.cpp 8d7285af40d9633608178ec239d3b8d65d3a2725 > src/checks/health_checker.cpp e70bd7936752613a4f92c70c4c61cd7cdf7c4ee5 > src/common/validation.hpp fee955139d9699caa651d9bbd03342f2eba8143f > src/common/validation.cpp b2548ad87b4227d6e498c49b5694acb362f6281b > src/master/validation.cpp e3f71be16ff21a1b4e10ad59846d9b06691bc1b9 > src/slave/validation.cpp c30a0ebf044ac81b087a8a599c7476bdc16bef18 > > Diff: https://reviews.apache.org/r/55954/diff/ > > > Testing > ------- > > Testing information at the end of this review chain. > > > Thanks, > > Greg Mann > >