> On May 11, 2017, 11:12 a.m., Reza Motamedi wrote: > > src/main/python/apache/aurora/client/cli/task.py > > Lines 224-225 (patched) > > <https://reviews.apache.org/r/59163/diff/1/?file=1715819#file1715819line224> > > > > Consider defending against some bash tricks, for instance what happens > > if the user set a path that has `~` or `../../../` to go up the dir chain. > > David McLaughlin wrote: > I think it's fine to delegate that to the scp command, no? If they have > the permissions to copy stuff outside of their container then you have bigger > problems :) > > Santhosh Kumar Shanmugham wrote: > Worth testing it on the vagrant box to see if you can break the chroot > jail. I am pretty sure Mesos already handles this. > > David McLaughlin wrote: > Or adding coverage to > src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh.
+1 - Santhosh Kumar ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59163/#review174699 ----------------------------------------------------------- On May 11, 2017, 10:13 a.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59163/ > ----------------------------------------------------------- > > (Updated May 11, 2017, 10:13 a.m.) > > > Review request for Aurora, David McLaughlin, Reza Motamedi, and Santhosh > Kumar. > > > Bugs: AURORA-1925 > https://issues.apache.org/jira/browse/AURORA-1925 > > > Repository: aurora > > > Description > ------- > > Added 'aurora task scp' command for copying/retrieving files to the sandbox > of a task instance. > > This command essentially mimics scp but expands task instances into their > respective user@host:path > For scp, the root for a task instance is the sandbox folder. > > Example usage: > From host to task sandbox folder: `aurora task scp ~/test.txt > cluster/role/env/job/instance:/` > From task sandbox folder to host: `aurora task scp > cluster/role/env/job/instance:/test.txt .` > From one task to another task: `aurora task scp > cluster/role/env/job/instance:/test.txt > cluster/role/env/job/instance:/some/dir/` > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/options.py > 11687033ac93226732dd1a742fb6d80f86a557f2 > src/main/python/apache/aurora/client/cli/task.py > 370dd7f730414641215dcd2da0b2a6037e6e1cf2 > src/test/python/apache/aurora/client/cli/test_task.py > 390993fc55d7aa717f39e25375b2d6cfe96242d0 > > > Diff: https://reviews.apache.org/r/59163/diff/1/ > > > Testing > ------- > > `./pants test src/test/python/apache/aurora/client/cli:cli` > ``` > 20:17:35 00:02 [chroot]============== test session starts > =============== > platform darwin -- Python 2.7.10 -- py-1.4.33 -- > pytest-2.6.4 > plugins: timeout, cov > collected 175 items > > > src/test/python/apache/aurora/client/cli/test_config_noun.py ... > src/test/python/apache/aurora/client/cli/test_context.py > ........ > src/test/python/apache/aurora/client/cli/test_version.py > . > src/test/python/apache/aurora/client/cli/test_quota.py > ..... > src/test/python/apache/aurora/client/cli/test_plugins.py > . > src/test/python/apache/aurora/client/cli/test_client.py > .. > src/test/python/apache/aurora/client/cli/test_sla.py > ..... > src/test/python/apache/aurora/client/cli/test_open.py > ..... > src/test/python/apache/aurora/client/cli/test_supdate.py > ....................................... > src/test/python/apache/aurora/client/cli/test_restart.py > .......... > src/test/python/apache/aurora/client/cli/test_status.py > ............. > src/test/python/apache/aurora/client/cli/test_add.py .... > src/test/python/apache/aurora/client/cli/test_diff.py .. > src/test/python/apache/aurora/client/cli/test_cron.py > .......... > > src/test/python/apache/aurora/client/cli/test_command_hooks.py .. > src/test/python/apache/aurora/client/cli/test_options.py > ...... > src/test/python/apache/aurora/client/cli/test_task.py > ........... > src/test/python/apache/aurora/client/cli/test_create.py > .............. > src/test/python/apache/aurora/client/cli/test_kill.py > ...................... > src/test/python/apache/aurora/client/cli/test_inspect.py > .... > > src/test/python/apache/aurora/client/cli/test_api_from_cli.py .. > > src/test/python/apache/aurora/client/cli/test_diff_formatter.py ...... > > ========== 175 passed in 26.42 seconds =========== > ``` > > I've also compiled it within the local cluster with Vagrant and used the > command to transfer a text file between the scheduler machine and job I > created. > > > Thanks, > > Jordan Ly > >
