]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
HNAS driver: Fix SSH and cluster_admin_ip0 bug
authorAdriano Rosso <adriano.rosso@fit-tecnologia.org.br>
Fri, 5 Feb 2016 17:41:01 +0000 (15:41 -0200)
committerAdriano Rosso <adriano.rosso@fit-tecnologia.org.br>
Fri, 12 Feb 2016 13:26:17 +0000 (11:26 -0200)
commit4f39141e329c1658d446fc335563630731f10884
tree7469691e29f1accf62beacf45127d42cad000f9d
parent04b2cdf9b5676fb49bdc2dcbd1115b9e34287f8c
HNAS driver: Fix SSH and cluster_admin_ip0 bug

HNAS driver has 2 configuration options:
- SSC (ssh_enabled: False ): requires a local utility package
installed to run the commands
- SSH (ssh_enabled: True): Runs the commands using the utility
package installed on HNAS

Currently, the HNAS driver is considering the "ssh_enabled" and the
"cluster_admin_ip0" tags to decide if the command to get the utility
package version should be ran locally or not. This is not correct
because the "cluster_admin_ip0" tag is not mandatory even if the SSH
mode is enabled.
So, if we have a situation that SSH is enabled and the
"cluster_admin_ip0" does not exist in the configuration, the driver
tries to run the command to get the utility package version locally
and its initialization breaks.
This patch makes the driver consider only the "ssh_enabled" tag
value to decide if the command to get the utility package version
is necessary.

Change-Id: I29fb5d7199e10eafee329dbd4ccef524245fbc28
Closes-Bug: 1543208
cinder/tests/unit/test_hitachi_hnas_backend.py
cinder/volume/drivers/hitachi/hnas_backend.py