Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-20 Thread Alexander Rojas
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48918/#review138543 --- Ship it! Ship It! - Alexander Rojas On June 20, 2016, 7:50

Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-20 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48918/#review138537 --- Ship it! Ship It! - Alexander Rukletsov On June 20, 2016,

Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-19 Thread Till Toenshoff
> On June 19, 2016, 10:40 p.m., Alexander Rukletsov wrote: > > src/slave/http.cpp, lines 1089-1093 > > > > > > How about this one? Nope, that one will not get invoked if the authorizer failed - `containers` will

Re: Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-19 Thread Alexander Rukletsov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48918/#review138488 --- src/slave/http.cpp

Review Request 48918: Removed explicit authorization results in Mesos.

2016-06-18 Thread Till Toenshoff
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48918/ --- Review request for mesos, Adam B, Alexander Rukletsov, Alexander Rojas,