Merge lp:~james-page/charms/precise/openstack-dashboard/theme-option into lp:~openstack-charmers/charms/precise/openstack-dashboard/ha-support

Proposed by James Page
Status: Merged
Merged at revision: 31
Proposed branch: lp:~james-page/charms/precise/openstack-dashboard/theme-option
Merge into: lp:~openstack-charmers/charms/precise/openstack-dashboard/ha-support
Diff against target: 41 lines (+11/-1)
3 files modified
config.yaml (+4/-0)
hooks/horizon-common (+1/-1)
hooks/horizon-relations (+6/-0)
To merge this branch: bzr merge lp:~james-page/charms/precise/openstack-dashboard/theme-option
Reviewer Review Type Date Requested Status
OpenStack Charmers Pending
Review via email: mp+154923@code.launchpad.net

Description of the change

Add support to turn on/off the ubuntu theme. Defaults on on.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.yaml'
--- config.yaml 2013-03-12 13:36:17 +0000
+++ config.yaml 2013-03-22 11:39:55 +0000
@@ -74,3 +74,7 @@
74 type: string74 type: string
75 default: "no"75 default: "no"
76 description: Show Django debug messages.76 description: Show Django debug messages.
77 ubuntu-theme:
78 type: string
79 default: "yes"
80 description: Use Ubuntu theme for the dashboard.
7781
=== modified file 'hooks/horizon-common'
--- hooks/horizon-common 2013-03-12 13:36:17 +0000
+++ hooks/horizon-common 2013-03-22 11:39:55 +0000
@@ -3,7 +3,7 @@
33
4CHARM="openstack-dashboard"4CHARM="openstack-dashboard"
55
6PACKAGES="openstack-dashboard openstack-dashboard-ubuntu-theme python-keystoneclient python-memcache memcached haproxy"6PACKAGES="openstack-dashboard python-keystoneclient python-memcache memcached haproxy"
7LOCAL_SETTINGS="/etc/openstack-dashboard/local_settings.py"7LOCAL_SETTINGS="/etc/openstack-dashboard/local_settings.py"
88
9if [[ -e "$CHARM_DIR/lib/openstack-common" ]] ; then9if [[ -e "$CHARM_DIR/lib/openstack-common" ]] ; then
1010
=== modified file 'hooks/horizon-relations'
--- hooks/horizon-relations 2013-03-13 11:08:28 +0000
+++ hooks/horizon-relations 2013-03-22 11:39:55 +0000
@@ -120,6 +120,12 @@
120 set_or_update DEBUG True120 set_or_update DEBUG True
121 fi121 fi
122122
123 if [ "$(config-get ubuntu-theme)" != "yes" ]; then
124 apt-get -y purge openstack-dashboard-ubuntu-theme || :
125 else
126 apt-get -y install openstack-dashboard-ubuntu-theme
127 fi
128
123 # Reconfigure Apache Ports129 # Reconfigure Apache Ports
124 configure_apache "80:70" "443:433"130 configure_apache "80:70" "443:433"
125 service apache2 restart131 service apache2 restart

Subscribers

People subscribed via source and target branches