-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43935/
-----------------------------------------------------------
(Updated 三月 28, 2016, 1:30 a.m.)
Review request for mesos, Greg Mann, Shuai Lin, and Michael Park.
Changes
-------
Fix michale's comment
Bugs: MESOS-4744
https://issues.apache.org/jira/browse/MESOS-4744
Repository: mesos
Description
-------
Allow setting role in mesos-execute.
Diffs (updated)
-----
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
