Bug 1129245

Summary: the enable_backends doesn't have a parameter and isn't commented out in cinder.conf
Product: Red Hat OpenStack Reporter: Yogev Rabl <yrabl>
Component: openstack-foreman-installerAssignee: Jason Guiditta <jguiditt>
Status: CLOSED WONTFIX QA Contact: nlevinki <nlevinki>
Severity: low Docs Contact:
Priority: low    
Version: 5.0 (RHEL 7)CC: jstransk, mburns, morazi, rhos-maint, sseago, yeylon, yrabl
Target Milestone: ---Keywords: ZStream
Target Release: Installer   
Hardware: All   
OS: All   
URL: https://trello.com/c/4aEzEymg/273-ux-enhancements
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-04-28 16:19:55 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:

Description Yogev Rabl 2014-08-12 11:27:57 UTC
Description of problem:
The enable_backends parameter shouldn't be left blank, it should have a name of a backend (LVM, NFS or Ceph) assigned to it with it create a new section, for an example: [LVM]
that holds all the driver parameters and settings.
Or the staypuft should comment out the enable_backends.

(the first option is preferred, for a single backend as well) 

Version-Release number of selected component (if applicable):
ruby193-rubygem-staypuft-0.2.2-1.el6ost.noarch

How reproducible:
100%

Actual results:
There's no value assigned to the enabled_backend parameter

Expected results:
1. set the used driver type (LVM/NFS/Ceph) with an addition section in the cinder.conf file
or
2. comment out the parameter

Additional info:

Comment 3 Mike Burns 2014-08-12 12:24:22 UTC
This should be handled in the quickstack modules, not in staypuft directly.

Comment 4 Jason Guiditta 2014-08-13 13:41:13 UTC
(In reply to Yogev Rabl from comment #0)
> Description of problem:
> The enable_backends parameter shouldn't be left blank, it should have a name
> of a backend (LVM, NFS or Ceph) assigned to it with it create a new section,
> for an example: [LVM]
> that holds all the driver parameters and settings.
> Or the staypuft should comment out the enable_backends.
> 
> (the first option is preferred, for a single backend as well) 
> 
> Version-Release number of selected component (if applicable):
> ruby193-rubygem-staypuft-0.2.2-1.el6ost.noarch
> 
> How reproducible:
> 100%
> 
> Actual results:
> There's no value assigned to the enabled_backend parameter
> 
> Expected results:
> 1. set the used driver type (LVM/NFS/Ceph) with an addition section in the
> cinder.conf file
This sounds like you have altered the cinder.conf directly?  If so, there is no way for puppet to pick up on that.  If not, what did you set in the staypuft UI for cinder configuration? Also, is this a non-HA setup?  Nova Networking or Neutron?

> or
> 2. comment out the parameter
> 
> Additional info:

Comment 5 Jiri Stransky 2014-08-13 15:55:45 UTC
What you're describing (use of enabled_backends and creating backend sections) happens when multiple_backends is set to true. When multiple_backends is false, cinder allows configuring the backend directly in the config file without having to create the sections. (Instead of using the sections, volume_driver parameter in cinder.conf is used to specify the backend driver. I think this is the original way before multiple backends were introduced). Here's an example how the config contents might look in more detail:

https://github.com/redhat-openstack/astapor/pull/238#issue-34555197

Is there something not working when using the volume_driver approach instead of enabled_backends?

In theory we could use the multi-backend approach even for single backend and get rid of the multiple_backends variable, but due to the way Openstack Puppet Modules handle config files it seems a bit safer not to do so. (Leftover backend sections can appear when switching multi-backend configurations. It shouldn't have a tangible impact though.)

Comment 6 Yogev Rabl 2014-08-14 07:05:52 UTC
Jason,
1. I haven't changed the cinder.conf manually, just reviewed it. 
2. I set the Cinder back end as Ceph and that's it, I didn't changed the advanced settings. 
3. It is a non-HA neutron setup. 

Jiri,
The Cinder can work with this configuration, the reason I opened this bug is because I see open variables in the config files as a practice we should avoid. 
To the second point, I've suggested the multiple backend approach as default because it will organize the cinder.conf which will make it easier to manage.

Comment 7 Arthur Berezin 2015-04-28 16:19:55 UTC
Refocusing engineering efforts on OSP7-Manager. 
Closing this BZ due to low priority and no customer cases attached.