diff --git a/.zuul.yaml b/.zuul.yaml index 67d4c24000..00129b5ca4 100644 --- a/.zuul.yaml +++ b/.zuul.yaml @@ -590,6 +590,9 @@ description: Ubuntu Bionic platform test nodeset: openstack-single-node-bionic voting: false + vars: + devstack_localrc: + CINDER_ISCSI_HELPER: tgtadm - job: name: devstack-async diff --git a/lib/cinder b/lib/cinder index f20631b56c..9c8d85cf59 100644 --- a/lib/cinder +++ b/lib/cinder @@ -88,15 +88,15 @@ CINDER_ENABLED_BACKENDS=${CINDER_ENABLED_BACKENDS:-lvm:lvmdriver-1} CINDER_VOLUME_CLEAR=${CINDER_VOLUME_CLEAR:-${CINDER_VOLUME_CLEAR_DEFAULT:-zero}} CINDER_VOLUME_CLEAR=$(echo ${CINDER_VOLUME_CLEAR} | tr '[:upper:]' '[:lower:]') -# Centos7 and OpenSUSE switched to using LIO and that's all that's supported, -# although the tgt bits are in EPEL and OpenSUSE we don't want that for CI +# Default to lioadm +CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-lioadm} + +# EL and SUSE should only use lioadm, we continue to allow Ubuntu based +# deployments to use tgtadm for specific jobs in the cinder-tempest-plugin if is_fedora || is_suse; then - CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-lioadm} if [[ ${CINDER_ISCSI_HELPER} != "lioadm" ]]; then die "lioadm is the only valid Cinder target_helper config on this platform" fi -else - CINDER_ISCSI_HELPER=${CINDER_ISCSI_HELPER:-tgtadm} fi # For backward compatibility