juju 1.17.2 doesn't pass command line options to ssh
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-core |
Fix Released
|
High
|
Dimiter Naydenov |
Bug Description
In juju 1.16.6, this works:
$ juju ssh ubuntu/0 -q ls /
bin boot dev etc home initrd.img lib lib64 lost+found media mnt opt proc root run sbin selinux srv sys tmp usr var vmlinuz
juju 1.16.6 calls ssh like this:
20647 execve(
The help says:
"Any extra parameters are passsed as extra parameters to the ssh command."
It doesn't work in juju 1.17.2:
$ juju ssh ubuntu/0 -q ls
Warning: Permanently added '10.55.32.243' (ECDSA) to the list of known hosts.
bash: -q: invalid option
Usage: bash [GNU long option] [option] ...
bash [GNU long option] [option] script-file ...
GNU long options:
--debug
--debugger
--dump-po-strings
--dump-strings
--help
--init-file
--login
--noediting
--noprofile
--norc
--posix
--protected
--rcfile
--restricted
--verbose
--version
Shell options:
-irsD or -c command or -O shopt_option (invocation only)
-abefhkmnptuvxBCHP or -o option
Connection to 10.55.32.243 closed.
ERROR exit status 2
juju 1.17.2 calls ssh like this:
20113 execve(
help says:
usage: juju ssh [options] <target> [<ssh args>...]
Any extra parameters are passsed as extra parameters to the ssh command.
Related branches
- Adam Collard (community): Needs Fixing
- Juju Engineering: Pending requested
-
Diff: 662 lines (+185/-87)12 files modifiedcmd/juju/addmachine.go (+1/-0)
cmd/juju/debughooks_test.go (+2/-2)
cmd/juju/scp.go (+34/-12)
cmd/juju/scp_test.go (+40/-11)
cmd/juju/ssh.go (+10/-8)
cmd/juju/ssh_test.go (+12/-18)
utils/ssh/run_test.go (+1/-1)
utils/ssh/ssh.go (+15/-7)
utils/ssh/ssh_gocrypto.go (+2/-1)
utils/ssh/ssh_gocrypto_test.go (+1/-1)
utils/ssh/ssh_openssh.go (+48/-14)
utils/ssh/ssh_test.go (+19/-12)
tags: | added: landscape |
tags: | added: docs regression ui |
Changed in juju-core: | |
status: | New → Triaged |
importance: | Undecided → High |
milestone: | none → 1.18.0 |
Changed in juju-core: | |
assignee: | nobody → Dimiter Naydenov (dimitern) |
Changed in juju-core: | |
status: | Triaged → In Progress |
Changed in juju-core: | |
milestone: | 1.18.0 → 1.17.4 |
Changed in juju-core: | |
status: | In Progress → Fix Committed |
Changed in juju-core: | |
status: | Fix Committed → Fix Released |