[dashboard] does not create the apache site properly on centos

Bug #1271193 reported by Darren Birkett
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Fix Released
High
Darren Birkett

Bug Description

On centos, the cookbook drops the following file:

/etc/httpd/conf.d/openstack-dashboard

However, the main httpd.conf includes the following:

Include /etc/httpd/conf.d/*.conf

So, it's not getting picked up because of the lack of .conf extension.

Tags: dashboard
Revision history for this message
Darren Birkett (darren-birkett) wrote :

We have two options to fix:

1) ensure file is created with correct extension on centos/rhel
2) use same a2ensite/a2dissite methods that we use on ubuntu. The sites-enabled/sites-disabled directory hierarchy is supported on current versions of centos/rhel (>=6.4 I think).

I prefer the latter as it is better code reusability and less maintenance.

Changed in openstack-chef:
assignee: nobody → Darren Birkett (darren-birkett)
Changed in openstack-chef:
importance: Undecided → High
summary: - openstack-dashboard on centos does not create the apache site properly
+ [dashboard] does not create the apache site properly on centos
tags: added: dashboard
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cookbook-openstack-dashboard (master)

Fix proposed to branch: master
Review: https://review.openstack.org/68367

Changed in openstack-chef:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-dashboard (master)

Reviewed: https://review.openstack.org/68367
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-dashboard/commit/?id=c8af3bd4fd9267ca58fedcb43e8263f0c1ab0dbb
Submitter: Jenkins
Branch: master

commit c8af3bd4fd9267ca58fedcb43e8263f0c1ab0dbb
Author: Darren Birkett <email address hidden>
Date: Wed Jan 22 12:36:02 2014 +0000

    creates openstack-dashboard site properly on centos/rhel

    Fixes: bug #1271193
    Implements: blueprint centos-rhel-for-dashboard

    On centos/rhel we now support the sites-enabled/sites-available style
    confs. This patch moves to use this approach on centos/rhel. It also
    fixes the above bug which was causing the site not to get loaded
    properly because it did not have the .conf extension. Also remove the
    rhel/centos specific tests for the openstack-dashboard site since
    rhel/centos now behaves the same as ubuntu in this regard.

    Note I am purposely ignoring fedora/others in this patch since the level
    of support for those is still tbd.

    Change-Id: I7c3339b7e3a5cd019b78dbb05425fef6fb344ebc

Changed in openstack-chef:
status: In Progress → Fix Released
Changed in openstack-chef:
status: Fix Committed → Fix Released
no longer affects: openstack-chef/havana
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.