----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62938/#review189436 -----------------------------------------------------------
Patch looks great! Reviews applied: [62938] Passed command: export OS='ubuntu:14.04' BUILDTOOL='autotools' COMPILER='gcc' CONFIGURATION='--verbose --disable-libtool-wrappers' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker-build.sh - Mesos Reviewbot On Oct. 26, 2017, 4:52 p.m., Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62938/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2017, 4:52 p.m.) > > > Review request for mesos, Eric Chung and Kevin Klues. > > > Bugs: MESOS-8012 > https://issues.apache.org/jira/browse/MESOS-8012 > > > Repository: mesos > > > Description > ------- > > This change allows users to use the CLI with a Mesos running in high > availability mode. The `zookeeper` field was already here before this > commit, with an `addresses` array and a `path` field. This change > only adds the backend to actually make it usable. > > Interacting with ZooKeeper requires a new dependency, kazoo, that has > been added to the list of requirements for the CLI. > > > Diffs > ----- > > src/python/cli_new/README.md a4b270d9b54cdd83bd62530fb44ddfaffb8a014b > src/python/cli_new/lib/cli/util.py dd109c09368e650b7d7dabd663c11bc2e3e5180a > src/python/cli_new/pip-requirements.txt > 7aeac344c47ccd2588fded44d7314db7abd85653 > > > Diff: https://reviews.apache.org/r/62938/diff/3/ > > > Testing > ------- > > Tested with a Mesos cluster running with 3 masters and 1 agent, run some CLI > commands successfully. > > > Thanks, > > Armand Grillet > >