> On Sept. 6, 2016, 6:24 p.m., Joseph Wu wrote: > > src/cli/execute.cpp, lines 111-121 > > <https://reviews.apache.org/r/51623/diff/1/?file=1490738#file1490738line111> > > > > There will be less code duplication if you made `--command` into an > > alias of `--commands`. You can parse a single command as a list with a > > single entry. > > > > (But this would allow you to specify multiple commands with the old > > flag, which may not be desireable.)
Yeah, so should we use a single `--command` option or multiple? I personally prefer only `--command` but not sure if it's what actually intended. - Abhishek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51623/#review147883 ----------------------------------------------------------- On Sept. 3, 2016, 8:36 p.m., Abhishek Dasgupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51623/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2016, 8:36 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 > ------- > > sThis patch updates mesos-execute to use LAUNCH_GROUP > for launching task groups. I made some assumption > in this patch, like newly introduced `--commands` > option in mesos-execute takes semicolon dimilited values > which will certainly not be the case in final solution. I > am open to suggestion what can we use as delimitter for > `--commands`. In this patch, I kept old `--command` option > as well and it is working as expected. Though new > `--commands` should be able to launch single as well > as multiple tasks and I hope in future we will stick > to one `commands` option. > > Suggestions are very welcome for further improvement. > > > Diffs > ----- > > src/cli/execute.cpp b752d057a3d86482ef1a4baaf31052469e38dc76 > > Diff: https://reviews.apache.org/r/51623/diff/ > > > Testing > ------- > > On Ubuntu 16.04: > sudo make -j4 > > and manually ran these commands: > **Successful** > mesos-execute --master=127.0.0.1:5050 --name="LetsCitar" > --containerizer=docker --docker_image="ubuntu" --command="echo hello" > > **Stuck after submitting task to agent as agent code for LAUNCH_GROUP is not > yet completed** > mesos-execute --master=127.0.0.1:5050 --name="LetsCitar" > --containerizer=docker --docker_image="ubuntu" --commands="echo hello" > > > Thanks, > > Abhishek Dasgupta > >