Configure Cinder to use lioadm on CentOS/RHEL 8

In CentOS/RHEL 8 there is no scsi-target-utils package, nor is it
available in EPEL. It is removed from kolla in [1]. In RHEL 7 and beyond
the LIO kernel subsystem can be used instead of the tgtd daemon.

This change removes support for the SCSI target daemon on CentOS/RHEL 8.
The 'tgtd' image is no longer available for CentOS/RHEL 8.

[1] https://review.openstack.org/#/c/613815/5

Change-Id: I718fc16cde2dd177b2a1c2f79b932426034897fe
Related: blueprint centos-rhel-8
This commit is contained in:
Mark Goddard 2019-01-17 12:19:55 +00:00 committed by Radosław Piliszek
parent 94def91405
commit 350bb171ef
5 changed files with 25 additions and 3 deletions

View File

@ -852,7 +852,7 @@ gnocchi_metric_datadir_volume: "gnocchi"
cinder_backend_ceph: "{{ enable_ceph }}"
cinder_backend_vmwarevc_vmdk: "no"
cinder_volume_group: "cinder-volumes"
cinder_target_helper: "tgtadm"
cinder_target_helper: "{{ 'lioadm' if ansible_os_family == 'RedHat' and ansible_distribution_major_version is version_compare('8', '>=') else 'tgtadm' }}"
# Valid options are [ '', redis, etcd ]
cinder_coordination_backend: "{{ 'redis' if enable_redis|bool else 'etcd' if enable_etcd|bool else '' }}"

View File

@ -17,3 +17,12 @@
- container_facts['iscsid'] is not defined
- inventory_hostname in groups[iscsi_services.iscsid.group]
- iscsi_services.iscsid.enabled | bool
- name: Check supported platforms for tgtd
fail:
msg:
The SCSI target daemon tgtd is not supported on CentOS/RHEL 8 and later
when:
- ansible_os_family == 'RedHat'
- ansible_distribution_major_version is version_compare('8', '>=')
- enable_tgtd | bool

View File

@ -0,0 +1,6 @@
---
upgrade:
- |
Support for the SCSI target daemon (``tgtd``) has been removed for
CentOS/RHEL 8. The default value of ``cinder_target_helper`` is now
``lioadm`` on CentOS/RHEL 8, but remains as ``tgtadm`` on other platforms.

View File

@ -234,6 +234,7 @@
chdir: "{{ kolla_ansible_src_dir }}"
environment:
BASE_DISTRO: "{{ base_distro }}"
BASE_DISTRO_MAJOR_VERSION: "{{ ansible_distribution_major_version }}"
INSTALL_TYPE: "{{ install_type }}"
BUILD_IMAGE: "{{ need_build_image }}"
TAG: "{{ build_image_tag }}"

View File

@ -40,11 +40,17 @@ function setup_config {
fi
if [[ $SCENARIO == "cinder-lvm" ]]; then
GATE_IMAGES+=",cinder,iscsid,tgtd"
GATE_IMAGES+=",cinder,iscsid"
if [[ $BASE_DISTRO != "centos" ]] || [[ $BASE_DISTRO_MAJOR_VERSION -eq 7 ]]; then
GATE_IMAGES+=",tgtd"
fi
fi
if [[ $SCENARIO == "zun" ]]; then
GATE_IMAGES+=",zun,kuryr,etcd,cinder,iscsid,tgtd"
GATE_IMAGES+=",zun,kuryr,etcd,cinder,iscsid"
if [[ $BASE_DISTRO != "centos" ]] || [[ $BASE_DISTRO_MAJOR_VERSION -eq 7 ]]; then
GATE_IMAGES+=",tgtd"
fi
fi
if [[ $SCENARIO == "scenario_nfv" ]]; then