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

Proposed by Adam Gandelman
Status: Merged
Merge reported by: Adam Gandelman
Merged at revision: not available
Proposed branch: lp:~gandelman-a/charms/precise/keystone/fixes
Merge into: lp:~charmers/charms/precise/keystone/trunk
Diff against target: 57 lines (+16/-3)
1 file modified
hooks/lib/openstack_common.py (+16/-3)
To merge this branch: bzr merge lp:~gandelman-a/charms/precise/keystone/fixes
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+136818@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/lib/openstack_common.py'
--- hooks/lib/openstack_common.py 2012-10-12 17:26:48 +0000
+++ hooks/lib/openstack_common.py 2012-11-29 00:13:25 +0000
@@ -10,7 +10,8 @@
10ubuntu_openstack_release = {10ubuntu_openstack_release = {
11 'oneiric': 'diablo',11 'oneiric': 'diablo',
12 'precise': 'essex',12 'precise': 'essex',
13 'quantal': 'folsom'13 'quantal': 'folsom',
14 'raring' : 'grizzly'
14}15}
1516
1617
@@ -60,6 +61,12 @@
60 ca_rel = ca_rel.split('%s-' % ubuntu_rel)[1].split('/')[0]61 ca_rel = ca_rel.split('%s-' % ubuntu_rel)[1].split('/')[0]
61 return ca_rel62 return ca_rel
6263
64 # Best guess match based on deb string provided
65 if src.startswith('deb'):
66 for k, v in openstack_codenames.iteritems():
67 if v in src:
68 return v
69
63def get_os_codename_version(vers):70def get_os_codename_version(vers):
64 '''Determine OpenStack codename from version number.'''71 '''Determine OpenStack codename from version number.'''
65 try:72 try:
@@ -132,7 +139,8 @@
132 return139 return
133 elif rel[:4] == "ppa:":140 elif rel[:4] == "ppa:":
134 src = rel141 src = rel
135 elif rel[:4] == "deb:":142 subprocess.check_call(["add-apt-repository", "-y", src])
143 elif rel[:3] == "deb":
136 l = len(rel.split('|'))144 l = len(rel.split('|'))
137 if l == 2:145 if l == 2:
138 src, key = rel.split('|')146 src, key = rel.split('|')
@@ -142,6 +150,9 @@
142 src = rel150 src = rel
143 else:151 else:
144 error_out("Invalid openstack-release: %s" % rel)152 error_out("Invalid openstack-release: %s" % rel)
153
154 with open('/etc/apt/sources.list.d/juju_deb.list', 'w') as f:
155 f.write(src)
145 elif rel[:6] == 'cloud:':156 elif rel[:6] == 'cloud:':
146 ubuntu_rel = lsb_release()['DISTRIB_CODENAME']157 ubuntu_rel = lsb_release()['DISTRIB_CODENAME']
147 rel = rel.split(':')[1]158 rel = rel.split(':')[1]
@@ -174,7 +185,9 @@
174185
175 src = "deb %s %s main" % (CLOUD_ARCHIVE_URL, pocket)186 src = "deb %s %s main" % (CLOUD_ARCHIVE_URL, pocket)
176 _import_key(CLOUD_ARCHIVE_KEY_ID)187 _import_key(CLOUD_ARCHIVE_KEY_ID)
188
189 with open('/etc/apt/sources.list.d/cloud-archive.list', 'w') as f:
190 f.write(src)
177 else:191 else:
178 error_out("Invalid openstack-release specified: %s" % rel)192 error_out("Invalid openstack-release specified: %s" % rel)
179193
180 subprocess.check_call(["add-apt-repository", "-y", src])

Subscribers

People subscribed via source and target branches