Merge lp:~gandelman-a/charms/oneiric/glance/custom_ppa into lp:~charmers/charms/oneiric/glance/trunk

Proposed by Adam Gandelman
Status: Merged
Approved by: Juan L. Negron
Approved revision: 14
Merge reported by: Juan L. Negron
Merged at revision: not available
Proposed branch: lp:~gandelman-a/charms/oneiric/glance/custom_ppa
Merge into: lp:~charmers/charms/oneiric/glance/trunk
Diff against target: 53 lines (+17/-5)
3 files modified
config.yaml (+1/-1)
hooks/glance-common (+15/-3)
hooks/glance-relations (+1/-1)
To merge this branch: bzr merge lp:~gandelman-a/charms/oneiric/glance/custom_ppa
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+85982@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Juan L. Negron (negronjl) wrote :

Reviewing this now.

-Juan

Revision history for this message
Juan L. Negron (negronjl) wrote :

Approved ... Merged.

-Juan

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.yaml'
--- config.yaml 2011-09-16 17:34:31 +0000
+++ config.yaml 2011-12-16 02:48:23 +0000
@@ -2,7 +2,7 @@
2 glance-release:2 glance-release:
3 default: distro3 default: distro
4 type: string4 type: string
5 description: Glance PPA from which to install. (distro, trunk, milestone, milestone-proposed)5 description: Glance PPA from which to install. (distro, trunk, milestone, milestone-proposed or ppa:somewhere/else)
6 api-config:6 api-config:
7 default: /etc/glance/glance-api.conf7 default: /etc/glance/glance-api.conf
8 type: string8 type: string
99
=== modified file 'hooks/glance-common'
--- hooks/glance-common 2011-12-02 23:33:07 +0000
+++ hooks/glance-common 2011-12-16 02:48:23 +0000
@@ -71,8 +71,20 @@
71 # Install from archive instead of PPA.71 # Install from archive instead of PPA.
72 [[ $PPA == "distro" ]] && return 072 [[ $PPA == "distro" ]] && return 0
73 . /etc/lsb-release73 . /etc/lsb-release
74 juju-log "glance: Setting up PPA for $PPA"
75 [[ -z $PPA ]] && return 074 [[ -z $PPA ]] && return 0
76 PPA_URL="deb http://ppa.launchpad.net/nova-core/$PPA/ubuntu $DISTRIB_CODENAME main"75 # if referenced by name, setup ppa to upstream PPAs
77 add-apt-repository "$PPA_URL" || exit 176 if [[ "$PPA" == "trunk" ]] ||
77 [[ "$PPA" == "milestone" ]] ||
78 [[ "$PPA" == "milestone-proposed" ]] ; then
79 juju-log "glance: Configuring installation from upstream PPA ($PPA)"
80 PPA_URL="deb http://ppa.launchpad.net/glance-core/$PPA/ubuntu $DISTRIB_CODENAME main"
81 add-apt-repository "$PPA_URL" || exit 1
82 return
83 fi
84 if [[ "${PPA:0:4}" == "ppa:" ]] ; then
85 juju-log "glance: Configuring installation from custom PPA ($PPA)"
86 add-apt-repository -y "$PPA" || exit 1
87 return
88 fi
89 juju-log "glance: No PPA specified. Falling back to installation from Ubuntu archive."
78}90}
7991
=== modified file 'hooks/glance-relations'
--- hooks/glance-relations 2011-12-02 23:33:07 +0000
+++ hooks/glance-relations 2011-12-16 02:48:23 +0000
@@ -10,7 +10,7 @@
10fi10fi
1111
12function install_hook {12function install_hook {
13 juju-log "Installing nova packages"13 juju-log "Installing glance packages"
14 apt-get -y install python-software-properties || exit 114 apt-get -y install python-software-properties || exit 1
1515
16 add_ppa16 add_ppa

Subscribers

People subscribed via source and target branches

to all changes: