]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix SAN generic driver ssh whitespaced commands
authorKallebe Monteiro <kallebe.monteiro@lsbd.ufc.br>
Fri, 6 Mar 2015 19:03:39 +0000 (16:03 -0300)
committerKallebe Monteiro <kallebe.monteiro@lsbd.ufc.br>
Wed, 18 Mar 2015 16:06:21 +0000 (13:06 -0300)
commit4b261e179e447a26012608a69da253bf7ecaafd1
tree0e28b54e541a2d8ea9edaf41b3ca133bcf4e3c96
parent35b1a03a4539d2e7c61d261ce800a241e603d65c
Fix SAN generic driver ssh whitespaced commands

SAN driver was sending commands over ssh with whitespaces between
every character resulting in commands like "u n a m e - s" instead
of "uname -s" (example from bug report).

As the bug reporter noted, the problem was that there were two joins
in the process of mounting the command. The first one got the
tuple and joined into a string. The second one joined a whitespace
between each character of the string.

Additionally, this behavior made utils.ssh_injection a bad check,
because it expected a list of args used in 'for arg in cmd_list' and
what it got was a single string. This resulted in the check being made
over the single characters instead of each arg.

All this is to explain why I chose to remove the first join instead of
the second one.

An unit test for this issue was created. SAN driver had no unit test.

Change-Id: I8e8624f2b82c26522936adf05e637a7bbe34a069
Closes-Bug: #1422604
cinder/tests/test_san.py [new file with mode: 0644]
cinder/volume/drivers/san/san.py