]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Parse out '@' in volume['host'] to do discovery
authorjohn-griffith <john.griffith@solidfire.com>
Wed, 27 Nov 2013 23:38:22 +0000 (16:38 -0700)
committerjohn-griffith <john.griffith@solidfire.com>
Wed, 27 Nov 2013 23:38:22 +0000 (16:38 -0700)
commitd869dee85aa9d32d1a397b954f6583d6bfa60c18
treeb293c077c191caf4cdd5fafca352274a26f7ab5b
parent79d807223674a23158c7c650b7c3717142aa04a9
Parse out '@' in volume['host'] to do discovery

The backup method of getting iscsi info is to use
iscsiadm discovery, however currently that method
just uses volume['host'] which in the case of
multi-backend will use "host@backend-name".

This will cause the discovery to fail of course, so
this change just parses out the '@' symbol if it's present
and avoids the problem in the first place.

This also beefs up the error logging and exception catching
a bit.

Parsing out the '@' symbol all the time should be safe as
the accepted valid chars for hostnames are digits, a-z and
hyphens.

Change-Id: Ic45a38bf4c56a4aec6847ab0d29e3b41d35bd3d2
Closes-Bug: #1250673
cinder/volume/driver.py