----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43935/#review124613 -----------------------------------------------------------
Looks great! A couple small comments below. src/cli/execute.cpp (lines 208 - 209) <https://reviews.apache.org/r/43935/#comment187226> Fits on one line. src/cli/execute.cpp (lines 215 - 216) <https://reviews.apache.org/r/43935/#comment187229> Could you add a comment here to explain the behavior that you and Guangya discussed previously, i.e. the behavior of `Resources::find` with respect to the framework's role? - Greg Mann On March 17, 2016, 1:39 a.m., Jian Qiu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43935/ > ----------------------------------------------------------- > > (Updated March 17, 2016, 1:39 a.m.) > > > Review request for mesos, Greg Mann, Shuai Lin, and Michael Park. > > > Bugs: MESOS-4744 > https://issues.apache.org/jira/browse/MESOS-4744 > > > Repository: mesos > > > Description > ------- > > Allow setting role in mesos-execute. > > > Diffs > ----- > > src/cli/execute.cpp ed42cb568f3d16856f48b3bbd354cb2b0fb83e8e > > Diff: https://reviews.apache.org/r/43935/diff/ > > > Testing > ------- > > make & make check > > start master. > ./bin/mesos-master.sh --work_dir=/tmp/mesos > > start slave. > ./bin/mesos-slave.sh --work_dir=/tmp/mesos --master=192.168.99.1:5050 > --resources="cpus:1;cpus(test):1;mem:7500;mem(test):7500" > > running mesos-execute without specifying role succeeds. > ./src/mesos-execute --master=192.168.99.1:5050 --name="test" --command="sleep > 10" --resources="cpus:1;mem:512" > > running mesos-execute with role test succeeds. > ./src/mesos-execute --master=192.168.99.1:5050 --name="test" --command="sleep > 10" --role="test" --resources="cpus:2;mem:512" > > running mesos-execute with role test1 fails. > ./src/mesos-execute --master=192.168.99.1:5050 --name="test" --command="sleep > 10" --role="test1" --resources="cpus:2;mem:512" > > > Thanks, > > Jian Qiu > >
