----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48264/#review136229 -----------------------------------------------------------
Patch looks great! Reviews applied: [48264] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On June 6, 2016, 8:19 a.m., Joerg Schad wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/48264/ > ----------------------------------------------------------- > > (Updated June 6, 2016, 8:19 a.m.) > > > Review request for mesos, Adam B, Till Toenshoff, and Vinod Kone. > > > Bugs: MESOS-5405 > https://issues.apache.org/jira/browse/MESOS-5405 > > > Repository: mesos > > > Description > ------- > > As MESOS-5405 changes the request.object > field to optional. We need to check that it only > matches an `ANY` ACL. > > > Diffs > ----- > > src/tests/authorization_tests.cpp 9e0f0b4f01ae6e466f6d44433a83edcebba4167a > > Diff: https://reviews.apache.org/r/48264/diff/ > > > Testing > ------- > > make check (OSX) + repeat new test 100 times > > > Thanks, > > Joerg Schad > >
