Merge lp:~stub/charms/precise/postgresql/cleanups into lp:charms/postgresql

Proposed by Stuart Bishop
Status: Merged
Merged at revision: 70
Proposed branch: lp:~stub/charms/precise/postgresql/cleanups
Merge into: lp:charms/postgresql
Prerequisite: lp:~stub/charms/precise/postgresql/charm-helpers
Diff against target: 40 lines (+7/-4)
2 files modified
config.yaml (+5/-2)
hooks/hooks.py (+2/-2)
To merge this branch: bzr merge lp:~stub/charms/precise/postgresql/cleanups
Reviewer Review Type Date Requested Status
Dave Cheney (community) Approve
Review via email: mp+190644@code.launchpad.net

Description of the change

Minor cleanups

To post a comment you must log in.
Revision history for this message
Dave Cheney (dave-cheney) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.yaml'
--- config.yaml 2013-10-03 12:14:03 +0000
+++ config.yaml 2013-10-11 12:58:25 +0000
@@ -289,7 +289,8 @@
289 description: >289 description: >
290 YAML map as e.g. "{ postgres/0: vol-0000010, postgres/1: vol-0000016 }".290 YAML map as e.g. "{ postgres/0: vol-0000010, postgres/1: vol-0000016 }".
291 Service units will raise a "configure-error" condition if no volume-map291 Service units will raise a "configure-error" condition if no volume-map
292 value is set for it - it's expected a human to set it properly to resolve it.292 value is set for it - it's expected a human to set it properly to
293 resolve it.
293 volume-dev-regexp:294 volume-dev-regexp:
294 type: string295 type: string
295 default: "/dev/vd[b-z]"296 default: "/dev/vd[b-z]"
@@ -322,7 +323,9 @@
322 default: ""323 default: ""
323 type: string324 type: string
324 description: |325 description: |
325 Extra archives to add with add-apt-repository(1).326 Extra archives to add, space separated. Supports ppa:, http:, cloud:
327 URIs, as well as other schemes and keywords supported by
328 charmhelpers.fetch.add_source() such as "proposed".
326 advisory_lock_restart_key:329 advisory_lock_restart_key:
327 default: 765330 default: 765
328 type: int331 type: int
329332
=== modified file 'hooks/hooks.py'
--- hooks/hooks.py 2013-10-10 10:51:12 +0000
+++ hooks/hooks.py 2013-10-11 12:58:25 +0000
@@ -1247,11 +1247,11 @@
1247 repos_added = False1247 repos_added = False
1248 for repo in extra_repos.split():1248 for repo in extra_repos.split():
1249 if repo not in extra_repos_added:1249 if repo not in extra_repos_added:
1250 run("add-apt-repository --yes '{}'".format(repo))1250 fetch.add_source(repo)
1251 extra_repos_added.add(repo)1251 extra_repos_added.add(repo)
1252 repos_added = True1252 repos_added = True
1253 if repos_added:1253 if repos_added:
1254 host.apt_update(fatal=True)1254 fetch.apt_update(fatal=True)
1255 local_state.save()1255 local_state.save()
12561256
12571257

Subscribers

People subscribed via source and target branches