Bug 1071423

Summary: on rhel7 cinder iscsi_helper should be set to "lioadm"
Product: [Community] RDO Reporter: Giulio Fidente <gfidente>
Component: openstack-puppet-modulesAssignee: Martin Magr <mmagr>
Status: CLOSED CURRENTRELEASE QA Contact: Ami Jeain <ajeain>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: apevec, yeylon
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: openstack-puppet-modules-2014.1-8.el7 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-04-30 11:09:00 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Quickfix LIO.patch for packstack-2013.2.1 dev956
none
Patch for openstack-puppet-modules-2014.1-1 RPM none

Description Giulio Fidente 2014-02-28 18:03:24 UTC 
Description of problem:
on rhel7 cinder iscsi_helper should be set to "lioadm", instead of "tgtadm"

puppet-cinder currently allows setting of iscsi_helper but doesn't have in cinder/manifests/volume/iscsi.pp a case for "lioadm"
Comment 1 Giulio Fidente 2014-02-28 18:05:52 UTC 
puppet-cinder seems to be currently forcing this to "tgtadm", see https://github.com/stackforge/puppet-cinder/blob/master/manifests/volume/iscsi.pp#L5
Comment 2 Alan Pevec 2014-02-28 23:50:25 UTC 
Created attachment 869258 [details]
Quickfix LIO.patch for packstack-2013.2.1 dev956

This is FYI, rebase to openstack-puppet-modules coming.
Comment 3 Alan Pevec 2014-03-01 00:02:33 UTC 
Created attachment 869260 [details]
Patch for openstack-puppet-modules-2014.1-1 RPM

First attempt but this would hard to maintain with current o-p-m source which include hash.