----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57950/#review170152 -----------------------------------------------------------
Fix it, then Ship it! I'll fix the issue while committing. include/mesos/authorizer/acls.proto Lines 404 (patched) <https://reviews.apache.org/r/57950/#comment242940> This is not an enum : ) include/mesos/authorizer/acls.proto Lines 409-411 (original), 410-412 (patched) <https://reviews.apache.org/r/57950/#comment242941> @arojas, isn't it time : )? - Alexander Rukletsov On March 27, 2017, 9:50 a.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57950/ > ----------------------------------------------------------- > > (Updated March 27, 2017, 9:50 a.m.) > > > Review request for mesos, Alexander Rukletsov, Alexander Rojas, and Vinod > Kone. > > > Repository: mesos > > > Description > ------- > > See https://reviews.apache.org/r/57386/#comment240626 for more details. > > > Diffs > ----- > > include/mesos/authorizer/acls.proto > e75e1879435f1c2bce47a86e9feebf9d051e969b > > > Diff: https://reviews.apache.org/r/57950/diff/1/ > > > Testing > ------- > > > Thanks, > > Gastón Kleiman > >
