----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55464/#review162361 -----------------------------------------------------------
Patch looks great! Reviews applied: [55676, 55722, 55677, 55678, 55679, 55464] 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 Jan. 19, 2017, 4:37 p.m., Gastón Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55464/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2017, 4:37 p.m.) > > > Review request for mesos, Adam B, Alexander Rukletsov, Anand Mazumdar, > Alexander Rojas, haosdent huang, and Vinod Kone. > > > Bugs: MESOS-6864 > https://issues.apache.org/jira/browse/MESOS-6864 > > > Repository: mesos > > > Description > ------- > > Made the Agent API able to handle containers nested at arbitrary levels. > > > Diffs > ----- > > src/slave/http.cpp ee7119179a6ddd935c3f43a618ef645619c305ee > src/slave/slave.cpp 205138add7a63289ff8ed81138f8b603828c748e > src/tests/api_tests.cpp ea62226fbc5dd0bcc5ea60d1bbcd3748f9367467 > > Diff: https://reviews.apache.org/r/55464/diff/ > > > Testing > ------- > > `GTEST_FILTER="" make -j 8 check` in macOS. > `make check` in Linux. > > I also did some manual testing using a proof of concept that makes the > `DefaultExecutor` leverage this change to perform CMD health checks. > > > Thanks, > > Gastón Kleiman > >