----------------------------------------------------------- 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. Changes ------- Addressed comments, added more validation. 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 (updated) ----- 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