<div dir="ltr">Actually, I was trying to fix it, and it is more broken than we realized. We no longer support multiple extra arguments to SCP. It always complains if there is one that isn't exactly the last one. (-o "foo" says it isn't accepted, -o"foo" is allowed, but you can't supply 2 of them, etc).<div>
It just needs to be fixed.</div><div>I think a better way is to allow "--", but just interpret each argument as potentially a target, and expand it if it has a clear MACHINE_OR_UNIT: at the beginning.</div><div>
<a href="https://bugs.launchpad.net/juju-core/+bug/1306208">https://bugs.launchpad.net/juju-core/+bug/1306208</a><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 10, 2014 at 7:05 PM, Aaron Bentley <span dir="ltr"><<a href="mailto:aaron.bentley@canonical.com" target="_blank">aaron.bentley@canonical.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
</div><div class="">On 14-04-10 08:23 AM, John Meinel wrote:<br>
> Note that copying off the all-machines log seems to be broken, the<br>
> lines:<br>
><br>
> juju --show-log scp -e test-release-aws -- -o<br>
> 'StrictHostKeyChecking no' -o 'UserKnownHostsFile /dev/null' -i<br>
> /var/lib/jenkins/cloud-city/staging-juju-rsa<br>
> 0:/var/log/juju/all-machines.log<br>
> /var/lib/jenkins/jobs/aws-upgrade/workspace/artifacts/all-machines-test-release-aws.log<br>
><br>
><br>
2014-04-10 03:40:59 INFO juju.cmd supercommand.go:297 running<br>
juju-1.18.0-precise-amd64 [gc]<br>
> 2014-04-10 03:40:59 INFO juju api.go:238 connecting to API<br>
> addresses: [<a href="http://ec2-54-86-4-94.compute-1.amazonaws.com:17070" target="_blank">ec2-54-86-4-94.compute-1.amazonaws.com:17070</a><br>
</div>> <<a href="http://ec2-54-86-4-94.compute-1.amazonaws.com:17070" target="_blank">http://ec2-54-86-4-94.compute-1.amazonaws.com:17070</a>>] 2014-04-10<br>
<div class="">> 03:40:59 INFO juju apiclient.go:114 state/api: dialing<br>
> "wss://<a href="http://ec2-54-86-4-94.compute-1.amazonaws.com:17070/" target="_blank">ec2-54-86-4-94.compute-1.amazonaws.com:17070/</a><br>
</div>> <<a href="http://ec2-54-86-4-94.compute-1.amazonaws.com:17070/" target="_blank">http://ec2-54-86-4-94.compute-1.amazonaws.com:17070/</a>>" 2014-04-10<br>
<div class="">> 03:40:59 INFO juju apiclient.go:124 state/api: connection<br>
> established 2014-04-10 03:40:59 ERROR juju.cmd supercommand.go:300<br>
> unexpected argument "-o"; extra arguments must be last<br>
><br>
><br>
> It looks like you have to spell it:<br>
><br>
> juju --show-log scp -e test-release-aws<br>
> 0:/var/log/juju/all-machines.log<br>
> /var/lib/jenkins/jobs/aws-upgrade/workspace/artifacts/all-machines-test-release-aws.log<br>
> -o 'StrictHostKeyChecking no' -o 'UserKnownHostsFile /dev/null' -i<br>
> /var/lib/jenkins/cloud-city/staging-juju-rsa<br>
><br>
> I don't know if "--" for SCP ever worked, but it appears 1.18 wants<br>
> a different spelling.<br>
<br>
</div>Using '--' to separate commands was the only way that used to work for<br>
juju scp.  Most all-machines.log that you'll find in CI bug reports<br>
were produced this way.<br>
<br>
If you passed -o directly, juju would complain that it was not a valid<br>
option, no matter where you put it.  This problem did not happen with<br>
juju ssh.<br>
<br>
It's nice that things are easier now, but it's a shame that the fix<br>
broke existing scripts.<br>
<div class=""><br>
Aaron<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1<br>
Comment: Using GnuPG with Thunderbird - <a href="http://www.enigmail.net/" target="_blank">http://www.enigmail.net/</a><br>
<br>
</div>iQEcBAEBAgAGBQJTRrMjAAoJEK84cMOcf+9hshsH/0338ET9mAJhcAHPd2pBdtzy<br>
9s4nXlARPrEH0/SW2IbmamSvLRRuNHRG/FrP/98fBsT3G9Db7099+ywgYhfzvtvN<br>
ucaE1Tu+1XiSi+ZFWW9CJP00JQP4CyPbpA9wsbHMbBPn3BBUO8+gkkmt2XbW1/1N<br>
MqkAgBRWuQDN7EGMYQLFhjIs6ipxv5wbtIRdEqdcChZbHsoKLNYSfdRQhdpq4ikR<br>
lkwAodnsMOP7KPGTyX8LUEYnNJYf1E516Hf7IaRTw875XC0Hk863fI58wBFG6fe+<br>
1DGM+l0C0H95Do4ExxgEq2xmONhL33/g7FzUVstaRNkm04xPvI4iypHMvZH08+A=<br>
=+ALO<br>
-----END PGP SIGNATURE-----<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Juju-dev mailing list<br>
<a href="mailto:Juju-dev@lists.ubuntu.com">Juju-dev@lists.ubuntu.com</a><br>
Modify settings or unsubscribe at: <a href="https://lists.ubuntu.com/mailman/listinfo/juju-dev" target="_blank">https://lists.ubuntu.com/mailman/listinfo/juju-dev</a><br>
</div></div></blockquote></div><br></div>