----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42334/#review114879 -----------------------------------------------------------
Ship it! src/master/validation.cpp (line 667) <https://reviews.apache.org/r/42334/#comment175714> We don't put \n in error message. I'll fix it for you. src/master/validation.cpp (line 669) <https://reviews.apache.org/r/42334/#comment175715> We don't put period in the end. I'll fix it for you. - Jie Yu On Jan. 16, 2016, 4:30 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42334/ > ----------------------------------------------------------- > > (Updated Jan. 16, 2016, 4:30 p.m.) > > > Review request for mesos, Jie Yu, Michael Park, and Vinod Kone. > > > Bugs: MESOS-4382 > https://issues.apache.org/jira/browse/MESOS-4382 > > > Repository: mesos > > > Description > ------- > > Changed `ReservationInfo.principal` from required to optional. > > In order to allow dynamic reservation without a principal, this field is > being changed to optional. However, the current patch alters the master to > invalidate any reserve operations that do not set this field. After a > deprecation cycle, the master will allow the field to be unset. > > > Diffs > ----- > > include/mesos/mesos.proto b12e0f3eff44d90ec01360fc08bf9e597d7ed9dd > include/mesos/v1/mesos.proto fa7e82e03b11cf6619a4f16e8e0fbbf755bf210c > src/master/validation.cpp c7cf56815fc743ff52ef423b23d78398ad1b35a3 > src/tests/master_validation_tests.cpp > fbf8fadbc04a7cbc60ee6091e0224339389b400f > > Diff: https://reviews.apache.org/r/42334/diff/ > > > Testing > ------- > > A new test was added, > `ReserveOperationValidationTest.ReservationInfoMissingPrincipal`, within the > master validation tests, to ensure that a `ReservationInfo` with no principal > will be invalidated. > > `make check` was used to test. > > > Thanks, > > Greg Mann > >
