----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59453/#review175820 -----------------------------------------------------------
Fix it, then Ship it! src/authorizer/local/authorizer.cpp Line 1342 (original), 1342 (patched) <https://reviews.apache.org/r/59453/#comment249154> Should be `acls.register_agents.agent` here. - Neil Conway On May 22, 2017, 3:53 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59453/ > ----------------------------------------------------------- > > (Updated May 22, 2017, 3:53 p.m.) > > > Review request for mesos, Adam B, Greg Mann, Michael Park, and Neil Conway. > > > Repository: mesos > > > Description > ------- > > Renames the field `RegisterAgent.agent` to `RegisterAgent.agents` in > order to come make it consistent with other ACLs. > > > Diffs > ----- > > include/mesos/authorizer/acls.proto > ae0b1ea2e6417d186b1606542d75f3a20e0811db > src/authorizer/local/authorizer.cpp > 89aaf4b712d337d519445c922606789c334e5101 > src/tests/authorization_tests.cpp 32aa6ac4db7854507127ea2fb88b3e92daa277c0 > src/tests/master_authorization_tests.cpp > e4233c19b1d9e3e2734259503d0daec4ce243667 > src/tests/script.cpp 791d331d6bdf178098b2ce0dfb185f9128632459 > > > Diff: https://reviews.apache.org/r/59453/diff/1/ > > > Testing > ------- > > make check > > > Thanks, > > Alexander Rojas > >
