Merge lp:~gandelman-a/charms/precise/openstack-dashboard/fixes into lp:~charmers/charms/precise/openstack-dashboard/trunk

Proposed by Adam Gandelman
Status: Merged
Merge reported by: Adam Gandelman
Merged at revision: not available
Proposed branch: lp:~gandelman-a/charms/precise/openstack-dashboard/fixes
Merge into: lp:~charmers/charms/precise/openstack-dashboard/trunk
Diff against target: 54 lines (+15/-2)
2 files modified
hooks/horizon-relations (+1/-1)
hooks/lib/openstack-common (+14/-1)
To merge this branch: bzr merge lp:~gandelman-a/charms/precise/openstack-dashboard/fixes
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+136820@code.launchpad.net

Description of the change

Some fixes for handling of repo URLs via 'openstack-origin' config. Courtesy of James Page, but I seem to have lost write access to these branches to merge myself.

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 'hooks/horizon-relations'
--- hooks/horizon-relations 2012-10-12 17:30:30 +0000
+++ hooks/horizon-relations 2012-11-29 00:14:30 +0000
@@ -11,7 +11,7 @@
11fi11fi
1212
13function install_hook {13function install_hook {
14 configure_install_source $(config-get openstack-origin)14 configure_install_source "$(config-get openstack-origin)"
15 apt-get update15 apt-get update
16 juju-log "$CHARM: Installing $PACKAGES."16 juju-log "$CHARM: Installing $PACKAGES."
17 DEBIAN_FRONTEND=noninteractive apt-get -y install $PACKAGES17 DEBIAN_FRONTEND=noninteractive apt-get -y install $PACKAGES
1818
=== modified file 'hooks/lib/openstack-common'
--- hooks/lib/openstack-common 2012-10-12 20:44:22 +0000
+++ hooks/lib/openstack-common 2012-11-29 00:14:30 +0000
@@ -78,7 +78,7 @@
78 juju-log "$CHARM No repository key specified"78 juju-log "$CHARM No repository key specified"
79 url="$src"79 url="$src"
80 fi80 fi
81 add-apt-repository -y "$url"81 echo $url > /etc/apt/sources.list.d/juju_deb.list
82 fi82 fi
83 return 083 return 0
84 fi84 fi
@@ -129,6 +129,7 @@
129 "oneiric") codename="diablo" ;;129 "oneiric") codename="diablo" ;;
130 "precise") codename="essex" ;;130 "precise") codename="essex" ;;
131 "quantal") codename="folsom" ;;131 "quantal") codename="folsom" ;;
132 "raring") codename="grizzly" ;;
132 esac133 esac
133 fi134 fi
134135
@@ -141,9 +142,21 @@
141 case "$ca_rel" in142 case "$ca_rel" in
142 "folsom"|"folsom/updates"|"folsom/proposed"|"folsom/staging")143 "folsom"|"folsom/updates"|"folsom/proposed"|"folsom/staging")
143 codename="folsom" ;;144 codename="folsom" ;;
145 "grizzly"|"grizzly/updates"|"grizzly/proposed"|"grizzy/staging")
146 codename="grizly" ;;
144 esac147 esac
145 fi148 fi
146 fi149 fi
150
151 # have a guess based on the deb string provided
152 if [[ "${rel:0:3}" == "deb" ]]; then
153 CODENAMES="diablo essex folsom grizzly"
154 for cname in $CODENAMES; do
155 if echo $rel | grep -q $cname; then
156 codename=$cname
157 fi
158 done
159 fi
147 echo $codename160 echo $codename
148}161}
149162

Subscribers

People subscribed via source and target branches