----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51623/#review149375 -----------------------------------------------------------
src/cli/execute.cpp (line 457) <https://reviews.apache.org/r/51623/#comment216961> s/mesos-execute-executor/default-executor/ src/cli/execute.cpp (lines 548 - 559) <https://reviews.apache.org/r/51623/#comment216962> just kill this block. not worth the complexity just to do a CHECK src/cli/execute.cpp (line 797) <https://reviews.apache.org/r/51623/#comment216967> this should not be set if "--task_group" is set. src/cli/execute.cpp (lines 857 - 870) <https://reviews.apache.org/r/51623/#comment216968> these should not be set if "--task_group" is set. src/cli/execute.cpp (lines 899 - 912) <https://reviews.apache.org/r/51623/#comment216969> this should not be set if "--task_group" is set. src/cli/execute.cpp (line 934) <https://reviews.apache.org/r/51623/#comment216970> new line - Vinod Kone On Sept. 18, 2016, 3:34 p.m., Abhishek Dasgupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51623/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2016, 3:34 p.m.) > > > Review request for mesos, Anand Mazumdar and Vinod Kone. > > > Bugs: MESOS-6096 > https://issues.apache.org/jira/browse/MESOS-6096 > > > Repository: mesos > > > Description > ------- > > Updated mesos-execute to support task groups. > > > Diffs > ----- > > src/cli/execute.cpp 525c89803ad1b29328420c5925a3e6045e487645 > > Diff: https://reviews.apache.org/r/51623/diff/ > > > Testing > ------- > > On Ubuntu 16.04: > sudo make -j4 > > and manually ran this command: > **Successful** > mesos-execute --master=127.0.0.1:5050 > --taskgroup=file:///home/abhishek/taskgroup.txt > > `/home/abhishek/taskgroup.txt:` > { > "tasks": [{ > "name": "sigmtdroid", > "task_id": { > "value": "sigmoid" > }, > "agent_id": { > "value": "" > }, > "resources": [{ > "name": "cpus", > "type": "SCALAR", > "scalar": { > "value": 1 > }, > "role": "*" > }, { > "name": "mem", > "type": "SCALAR", > "scalar": { > "value": 32 > }, > "role": "*" > }], > "command": { > "value": "sleep 1000" > }, > "volumes": [{ > "container_path": "/mnt/volume", > "mode": "RW", > "source": { > "docker_volume": { > "driver": "volume_driver", > "docker_options": { > "parameter": [{ > "key": "key", > "value": "value" > }] > }, > "name": "volume_name" > }, > "type": "DOCKER_VOLUME" > } > }] > }, { > "name": "sigmteroid", > "task_id": { > "value": "sigmoid2" > }, > "agent_id": { > "value": "" > }, > "resources": [{ > "name": "cpus", > "type": "SCALAR", > "scalar": { > "value": 2 > }, > "role": "*" > }, { > "name": "mem", > "type": "SCALAR", > "scalar": { > "value": 32 > }, > "role": "*" > }], > "command": { > "value": "sleep 1000" > }, > "volumes": [{ > "container_path": "/mnt/volume", > "mode": "RW", > "source": { > "docker_volume": { > "driver": "volume_driver", > "docker_options": { > "parameter": [{ > "key": "key", > "value": "value" > }] > }, > "name": "volume_name" > }, > "type": "DOCKER_VOLUME" > } > }] > }] > } > > > Thanks, > > Abhishek Dasgupta > >
