> On Sept. 16, 2015, 12:31 a.m., Bill Farner wrote:
> > src/main/python/apache/aurora/client/api/command_runner.py, line 94
> > <https://reviews.apache.org/r/38326/diff/1/?file=1068938#file1068938line94>
> >
> >     This patch uses mutable lists in several places, which i feel is a 
> > divergence from general prefernce for immutability.  I don't think it 
> > hinders readability at all to assign these lists once and not mutate them.

The options param is a list. This is the simplest solution to merge it right 
into the ssh_command without inserting it as a list.


- Maxim


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38326/#review99130
-----------------------------------------------------------


On Sept. 11, 2015, 11:31 p.m., Maxim Khutornenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38326/
> -----------------------------------------------------------
> 
> (Updated Sept. 11, 2015, 11:31 p.m.)
> 
> 
> Review request for Aurora, Bill Farner and Zameer Manji.
> 
> 
> Bugs: AURORA-1491
>     https://issues.apache.org/jira/browse/AURORA-1491
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Adding ssh options into "aurora task" commands.
> 
> 
> Diffs
> -----
> 
>   src/main/python/apache/aurora/client/api/command_runner.py 
> c7238e274e53138187c2fe6fe5f14b7ae5f43ba2 
>   src/main/python/apache/aurora/client/cli/options.py 
> 41b13d6f0e1ed355ea8b958b875c20f065349465 
>   src/main/python/apache/aurora/client/cli/task.py 
> d1f2568ac0afdd95c65523fde41f0dd16670a7a8 
>   src/test/python/apache/aurora/client/cli/test_task.py 
> 3ad0b70a7d918055ffee34f593d108a28de6e9f9 
> 
> Diff: https://reviews.apache.org/r/38326/diff/
> 
> 
> Testing
> -------
> 
> In vagrant:
> ```
> vagrant@aurora:~$ aurora task run -v --ssh-user=vagrant --ssh-options='-v -k' 
> --executor-sandbox devcluster/www-data/prod/hello 'ls'
> DEBUG] Command=(['task', 'run', '-v', '--ssh-user=vagrant', '--ssh-options=-v 
> -k', '--executor-sandbox', 'devcluster/www-data/prod/hello', 'ls'])
> DEBUG] Using auth module: 
> <apache.aurora.common.auth.auth_module.InsecureAuthModule object at 
> 0x7f05d2b88250>
> DEBUG] Running command: ['ssh', '-n', '-q', '-v', '-k', 
> 'vagrant@192.168.33.7', u'cd 
> /var/lib/mesos/slaves/*/frameworks/*/executors/thermos-1442013544190-www-data-prod-hello-0-fe7fa2f1-e21b-4cc3-9107-0777da35537e/runs/latest;ls']
> 192.168.33.7:  OpenSSH_6.6.1, OpenSSL 1.0.1f 6 Jan 2014
> 192.168.33.7:  debug1: Reading configuration data /etc/ssh/ssh_config
> 192.168.33.7:  debug1: /etc/ssh/ssh_config line 19: Applying options for *
> 192.168.33.7:  debug1: /etc/ssh/ssh_config line 57: Applying options for *
> 192.168.33.7:  debug1: Connecting to 192.168.33.7 [192.168.33.7] port 22.
> 192.168.33.7:  debug1: Connection established.
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_rsa type 1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_rsa-cert type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_dsa type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_dsa-cert type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_ecdsa type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_ecdsa-cert type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_ed25519 type -1
> 192.168.33.7:  debug1: identity file /home/vagrant/.ssh/id_ed25519-cert type 
> -1
> 192.168.33.7:  debug1: Enabling compatibility mode for protocol 2.0
> 192.168.33.7:  debug1: Local version string SSH-2.0-OpenSSH_6.6.1p1 
> Ubuntu-2ubuntu2
> 192.168.33.7:  debug1: Remote protocol version 2.0, remote software version 
> OpenSSH_6.6.1p1 Ubuntu-2ubuntu2
> 192.168.33.7:  debug1: match: OpenSSH_6.6.1p1 Ubuntu-2ubuntu2 pat 
> OpenSSH_6.6.1* compat 0x04000000
> 192.168.33.7:  debug1: SSH2_MSG_KEXINIT sent
> 192.168.33.7:  debug1: SSH2_MSG_KEXINIT received
> 192.168.33.7:  debug1: kex: server->client aes128-ctr 
> hmac-md5-...@openssh.com none
> 192.168.33.7:  debug1: kex: client->server aes128-ctr 
> hmac-md5-...@openssh.com none
> 192.168.33.7:  debug1: sending SSH2_MSG_KEX_ECDH_INIT
> 192.168.33.7:  debug1: expecting SSH2_MSG_KEX_ECDH_REPLY
> 192.168.33.7:  debug1: Server host key: ECDSA 
> 46:15:09:58:38:39:76:02:68:e1:c2:43:1f:70:bf:6e
> 192.168.33.7:  Warning: Permanently added '192.168.33.7' (ECDSA) to the list 
> of known hosts.
> 192.168.33.7:  debug1: ssh_ecdsa_verify: signature correct
> 192.168.33.7:  debug1: SSH2_MSG_NEWKEYS sent
> 192.168.33.7:  debug1: expecting SSH2_MSG_NEWKEYS
> 192.168.33.7:  debug1: SSH2_MSG_NEWKEYS received
> 192.168.33.7:  debug1: Roaming not allowed by server
> 192.168.33.7:  debug1: SSH2_MSG_SERVICE_REQUEST sent
> 192.168.33.7:  debug1: SSH2_MSG_SERVICE_ACCEPT received
> 192.168.33.7:  debug1: Authentications that can continue: publickey,password
> 192.168.33.7:  debug1: Next authentication method: publickey
> 192.168.33.7:  debug1: Offering RSA public key: /home/vagrant/.ssh/id_rsa
> 192.168.33.7:  debug1: Server accepts key: pkalg ssh-rsa blen 279
> 192.168.33.7:  debug1: key_parse_private2: missing begin marker
> 192.168.33.7:  debug1: read PEM private key done: type RSA
> 192.168.33.7:  debug1: Authentication succeeded (publickey).
> 192.168.33.7:  Authenticated to 192.168.33.7 ([192.168.33.7]:22).
> 192.168.33.7:  debug1: channel 0: new [client-session]
> 192.168.33.7:  debug1: Requesting no-more-sessi...@openssh.com
> 192.168.33.7:  debug1: Entering interactive session.
> 192.168.33.7:  debug1: Sending environment.
> 192.168.33.7:  debug1: Sending env LANG = en_US.UTF-8
> 192.168.33.7:  debug1: Sending env LC_CTYPE = en_US.UTF-8
> 192.168.33.7:  debug1: Sending command: cd 
> /var/lib/mesos/slaves/*/frameworks/*/executors/thermos-1442013544190-www-data-prod-hello-0-fe7fa2f1-e21b-4cc3-9107-0777da35537e/runs/latest;ls
> 192.168.33.7:  debug1: client_input_channel_req: channel 0 rtype exit-status 
> reply 0
> 192.168.33.7:  checkpoints
> 192.168.33.7:  __main__.log
> 192.168.33.7:  sandbox
> 192.168.33.7:  stderr
> 192.168.33.7:  stdout
> 192.168.33.7:  task.json
> 192.168.33.7:  thermos_executor.pex
> 192.168.33.7:  thermos_runner.aurora.root.log.DEBUG.20150911-231905.16314
> 192.168.33.7:  thermos_runner.aurora.root.log.ERROR.20150911-231905.16314
> 192.168.33.7:  thermos_runner.aurora.root.log.FATAL.20150911-231905.16314
> 192.168.33.7:  thermos_runner.aurora.root.log.INFO.20150911-231905.16314
> 192.168.33.7:  thermos_runner.aurora.root.log.WARNING.20150911-231905.16314
> 192.168.33.7:  thermos_runner.DEBUG
> 192.168.33.7:  thermos_runner.ERROR
> 192.168.33.7:  thermos_runner.FATAL
> 192.168.33.7:  thermos_runner.INFO
> 192.168.33.7:  thermos_runner.pex
> 192.168.33.7:  thermos_runner.WARNING
> 192.168.33.7:  debug1: channel 0: free: client-session, nchannels 1
> 192.168.33.7:  debug1: fd 1 clearing O_NONBLOCK
> 192.168.33.7:  Transferred: sent 3568, received 2996 bytes, in 0.2 seconds
> 192.168.33.7:  Bytes per second: sent 22735.7, received 19090.8
> 192.168.33.7:  debug1: Exit status 0
> DEBUG] Command terminated successfully
> 
> ```
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>

Reply via email to