----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42334/#review114862 -----------------------------------------------------------
include/mesos/mesos.proto (lines 575 - 576) <https://reviews.apache.org/r/42334/#comment175702> Does "username of the operator" is a MUST? I think that a end user can reserve resources with any principal? src/master/validation.cpp (lines 663 - 664) <https://reviews.apache.org/r/42334/#comment175703> I saw that most comments including some verison info if we want to deprecate sth in one release. Please refer to https://github.com/apache/mesos/blob/master/src/master/master.cpp#L909-L910 for detail. - Guangya Liu On 一月 15, 2016, 11:06 p.m., Greg Mann wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42334/ > ----------------------------------------------------------- > > (Updated 一月 15, 2016, 11:06 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 > >
