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
1=== modified file 'config.yaml'
2--- config.yaml 2013-03-12 13:36:17 +0000
3+++ config.yaml 2013-03-22 11:39:55 +0000
4@@ -74,3 +74,7 @@
5 type: string
6 default: "no"
7 description: Show Django debug messages.
8+ ubuntu-theme:
9+ type: string
10+ default: "yes"
11+ description: Use Ubuntu theme for the dashboard.
12
13=== modified file 'hooks/horizon-common'
14--- hooks/horizon-common 2013-03-12 13:36:17 +0000
15+++ hooks/horizon-common 2013-03-22 11:39:55 +0000
16@@ -3,7 +3,7 @@
17
18 CHARM="openstack-dashboard"
19
20-PACKAGES="openstack-dashboard openstack-dashboard-ubuntu-theme python-keystoneclient python-memcache memcached haproxy"
21+PACKAGES="openstack-dashboard python-keystoneclient python-memcache memcached haproxy"
22 LOCAL_SETTINGS="/etc/openstack-dashboard/local_settings.py"
23
24 if [[ -e "$CHARM_DIR/lib/openstack-common" ]] ; then
25
26=== modified file 'hooks/horizon-relations'
27--- hooks/horizon-relations 2013-03-13 11:08:28 +0000
28+++ hooks/horizon-relations 2013-03-22 11:39:55 +0000
29@@ -120,6 +120,12 @@
30 set_or_update DEBUG True
31 fi
32
33+ if [ "$(config-get ubuntu-theme)" != "yes" ]; then
34+ apt-get -y purge openstack-dashboard-ubuntu-theme || :
35+ else
36+ apt-get -y install openstack-dashboard-ubuntu-theme
37+ fi
38+
39 # Reconfigure Apache Ports
40 configure_apache "80:70" "443:433"
41 service apache2 restart

Subscribers

People subscribed via source and target branches