----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52042/#review151505 -----------------------------------------------------------
Patch looks great! Reviews applied: [52042] 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 Oct. 5, 2016, 12:31 p.m., Alexander Rojas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52042/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2016, 12:31 p.m.) > > > Review request for mesos and Adam B. > > > Repository: mesos > > > Description > ------- > > If attempts to use the pailer to access sandboxes and/or mesos logs fail > because the user is unauthorized, an error telling the user he is > unauthorized is shown to the client instead of showing the misguiding > _Failed to initialize... retrying_. It also disables retry in case the > client is unauthorized. > > > Diffs > ----- > > src/webui/master/static/js/jquery.pailer.js > 79c3c0dad59c2dde78913116bb08383f7a6c4d5e > > Diff: https://reviews.apache.org/r/52042/diff/ > > > Testing > ------- > > `make check` > > and manual tests with the following ACLs > > ```json > { > "permissive": true, > "access_mesos_logs" : [ > { > "principals" : { "values" : ["foo"] }, > "logs" : { "type" : "NONE" } > } > ], > "access_sandboxes" : [ > { > "principals" : { "values" : ["foo"] }, > "users" : { "type" : "NONE" } > } > ] > } > > ``` > > and credentials: > > ``` > foo bar > baz bar > ``` > > > Thanks, > > Alexander Rojas > >
