----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59163/#review174699 -----------------------------------------------------------
src/main/python/apache/aurora/client/cli/task.py Lines 167 (patched) <https://reviews.apache.org/r/59163/#comment247911> Help should be a bit longer here. What does it accept? aurora task scp zone/role/env/job/instance:from.txt to.txt and aurora task scp from.txt zone/role/env/job/instance:to.txt ? src/main/python/apache/aurora/client/cli/task.py Lines 170 (patched) <https://reviews.apache.org/r/59163/#comment247905> nitpicking... staticmethod? src/main/python/apache/aurora/client/cli/task.py Lines 179 (patched) <https://reviews.apache.org/r/59163/#comment247907> nitpicking... staticmethod? I think when you are not really accessing instance data, you relly want a static method. src/main/python/apache/aurora/client/cli/task.py Lines 224-225 (patched) <https://reviews.apache.org/r/59163/#comment247912> 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. - Reza Motamedi On May 11, 2017, 5:13 p.m., Jordan Ly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59163/ > ----------------------------------------------------------- > > (Updated May 11, 2017, 5:13 p.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 > >
