Bug 1061689

Summary: Horizon SSL is disabled by Nagios configuration via packstack
Product: [Community] RDO Reporter: Nir Magnezi <nmagnezi>
Component: openstack-packstackAssignee: RHOS Maint <rhos-maint>
Status: CLOSED CURRENTRELEASE QA Contact: yeylon <yeylon>
Severity: medium Docs Contact:
Priority: medium    
Version: unspecifiedCC: aortega, derekh, ichavero, lhh, srevivo, yeylon
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: openstack-packstack-2014.2-0.17.dev1462.gbb05296.fc22 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1054498 Environment:
Last Closed: 2016-03-30 23:08:19 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:
Bug Depends On: 966094, 1054498    
Bug Blocks:    

Description Nir Magnezi 2014-02-05 12:47:01 UTC 
Bug reproduced with openstack-packstack-2013.2.1-0.29.dev956.el6.noarch in RDO Icehouse

+++ This bug was initially created as a clone of Bug #1054498 +++

Description of problem:

Nagios puppet modules disable SSL after the Horizon module enables it.


Version-Release number of selected component (if applicable): openstack-packstack-2013.2.1-0.22.dev956.el6ost.noarch


How reproducible: 100%

Steps to Reproduce:
1. packstack --os-horizon-ssl=y --allinone
2. (wait)

Actual results:

After completion, SSL for Apache is disabled - thus, Horizon SSL does not work.


Expected results:

Unknown.  The Nagios module reset the Apache configuration to largely default, clearing out the Apache SSL configuration performed by the Horizon module.


Additional info:

packstack --os-horizon-ssl=y --nagios-install=n --allinone

...works fine...

--- Additional comment from Ivan Chavero on 2014-01-20 13:42:38 EST ---

Modifying the nagios template so it uses the SSL configuration given by horizon.

--- Additional comment from Ivan Chavero on 2014-01-24 16:56:14 EST ---

patch submitted to gerrit code review: https://review.openstack.org/#/c/68557/

The nagios template uses the apache class, this
class overwrites the current configuration thus
deleting the horizon configurations. By checking
if horizon is installed this patch avoids the use
of the apache puppeet class in the nagios template.

--- Additional comment from Ivan Chavero on 2014-01-27 02:16:31 EST ---

The nagios template uses the apache class, this
class overwrites the current configuration thus
deleting the horizon configurations. By checking
if horizon is installed this patch avoids the use
of the apache puppeet class in the nagios template.