Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/ --- (Updated May 27, 2016, 4:50 a.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-26 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/#review135090 --- Ship it! This ship-it is barring your comments. Thanks so much

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-26 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/#review135077 --- src/tests/master_validation_tests.cpp (line 36)

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-25 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/ --- (Updated May 25, 2016, 6:08 p.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-25 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/#review134766 --- Fix it, then Ship it! src/tests/master_validation_tests.cpp

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-23 Thread Greg Mann
> On May 23, 2016, 2:06 p.m., Bernd Mathiske wrote: > > src/tests/master_validation_tests.cpp, line 454 > > > > > > Where are we setting the principal for DiskInfo.Persistence? > > > > How is this block of

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-23 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/ --- (Updated May 24, 2016, 4:44 a.m.) Review request for mesos, Bernd Mathiske,

Re: Review Request 47528: Updated validation tests with creator principal.

2016-05-23 Thread Bernd Mathiske
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47528/#review134346 --- src/tests/master_validation_tests.cpp (line 451)