ISCSI services should be started for LVM backend
Change-Id: I34f36b91b7bb9c79f2bc9a57d6f7a76a50e0e131
Related-Bug: PROD-22900 (PROD:22900)
diff --git a/cinder/map.jinja b/cinder/map.jinja
index 79dff75..5bf2863 100644
--- a/cinder/map.jinja
+++ b/cinder/map.jinja
@@ -77,7 +77,9 @@
'BaseDefaults': default_params,
'Debian': {
'pkgs': ['cinder-volume', 'lvm2', 'sysfsutils', 'sg3-utils', 'python-cinder','python-mysqldb','p7zip', 'gettext-base', 'python-memcache', 'python-pycadf'],
+ 'iscsi_pkgs': ['iscsitarget', 'open-iscsi', 'iscsitarget-dkms'],
'services': ['cinder-volume'],
+ 'iscsi_services': ['iscsitarget', 'open-iscsi'],
'wipe_method': 'none',
'notification': False,
'audit': {
@@ -106,7 +108,9 @@
},
'RedHat': {
'pkgs': ['openstack-cinder', 'python-cinder', 'lvm2', 'sysfsutils', 'sg3_utils', 'device-mapper-multipath', 'device-mapper-multipath-libs', 'python-pycadf'],
+ 'iscsi_pkgs': ['iscsitarget', 'open-iscsi', 'iscsitarget-dkms'],
'services': ['openstack-cinder-volume'],
+ 'iscsi_services': ['iscsitarget', 'open-iscsi'],
'wipe_method': 'none',
'notification': False,
'audit': {
diff --git a/cinder/volume.sls b/cinder/volume.sls
index 080466f..6a466ff 100644
--- a/cinder/volume.sls
+++ b/cinder/volume.sls
@@ -191,14 +191,11 @@
{%- endif %}
-{%- if backend.engine in ['iscsi' , 'hp_lefthand'] %}
+{%- if backend.engine in ['iscsi' , 'hp_lefthand', 'lvm'] %}
cinder_iscsi_packages_{{ loop.index }}:
pkg.installed:
- - names:
- - iscsitarget
- - open-iscsi
- - iscsitarget-dkms
+ - names: {{ volume.iscsi_pkgs }}
- require:
- pkg: cinder_volume_packages
@@ -211,9 +208,7 @@
cinder_scsi_service:
service.running:
- - names:
- - iscsitarget
- - open-iscsi
+ - names: {{ volume.iscsi_services }}
- enable: true
{%- if grains.get('noservices') %}
- onlyif: /bin/false