Browse code

Merge "Switch to lioadm for centos cinder"

Jenkins authored on 2017/06/29 08:22:30
Showing 2 changed files
... ...
@@ -1,4 +1,5 @@
1 1
 iscsi-initiator-utils
2 2
 lvm2
3 3
 qemu-img
4
-scsi-target-utils # NOPRIME
4
+scsi-target-utils # not:rhel7,f24,f25 NOPRIME
5
+targetcli # dist:rhel7,f24,f25 NOPRIME
5 6
\ No newline at end of file
... ...
@@ -95,7 +95,16 @@ CINDER_VOLUME_CLEAR=$(echo ${CINDER_VOLUME_CLEAR} | tr '[:upper:]' '[:lower:]')
95 95
 # https://bugs.launchpad.net/cinder/+bug/1180976
96 96
 CINDER_PERIODIC_INTERVAL=${CINDER_PERIODIC_INTERVAL:-60}
97 97
 
98
-CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-tgtadm}
98
+# Centos7 switched to using LIO and that's all that's supported,
99
+# although the tgt bits are in EPEL we don't want that for CI
100
+if is_fedora; then
101
+    CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-lioadm}
102
+    if [[ ${CINDER_ISCSI_HELPER} != "lioadm" ]]; then
103
+        die "lioadm is the only valid Cinder iscsi_helper config on this platform"
104
+    fi
105
+else
106
+    CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-tgtadm}
107
+fi
99 108
 
100 109
 # Toggle for deploying Cinder under HTTPD + mod_wsgi
101 110
 CINDER_USE_MOD_WSGI=${CINDER_USE_MOD_WSGI:-False}
... ...
@@ -435,12 +444,10 @@ function init_cinder {
435 435
 function install_cinder {
436 436
     git_clone $CINDER_REPO $CINDER_DIR $CINDER_BRANCH
437 437
     setup_develop $CINDER_DIR
438
-    if [ "$CINDER_ISCSI_HELPER" = "tgtadm" ]; then
439
-        if is_fedora; then
440
-            install_package scsi-target-utils
441
-        else
442
-            install_package tgt
443
-        fi
438
+    if [[ "$CINDER_ISCSI_HELPER" == "tgtadm" ]]; then
439
+        install_package tgt
440
+    elif [[ "$CINDER_ISCI_HELPER" == "lioadm" ]]; then
441
+        install_package targetcli
444 442
     fi
445 443
 
446 444
     if [ "$CINDER_USE_MOD_WSGI" == "True" ]; then