----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43935/#review125329 -----------------------------------------------------------
Fix it, then Ship it! src/cli/execute.cpp (line 214) <https://reviews.apache.org/r/43935/#comment188100> s/sepcified/specified/ - Greg Mann On March 22, 2016, 2:57 a.m., Jian Qiu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43935/ > ----------------------------------------------------------- > > (Updated March 22, 2016, 2:57 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 > >
