Merge ~fginther/postgresql-charm:master into postgresql-charm:master

Proposed by Francis Ginther
Status: Merged
Merged at revision: d715cd1432c2001eb033fc487e5a6cf01a99fdf6
Proposed branch: ~fginther/postgresql-charm:master
Merge into: postgresql-charm:master
Diff against target: 17 lines (+4/-4)
1 file modified
reactive/postgresql/wal_e.py (+4/-4)
Reviewer Review Type Date Requested Status
Stuart Bishop (community) Approve
Review via email: mp+303442@code.launchpad.net

Commit message

Remove the addition of ppa:stub/pgcharm by the charm itself to allow use of wal-e in restricted environments.

Description of the change

Remove the addition of ppa:stub/pgcharm by the charm itself to allow use of wal-e in restricted environments.

In a production environment, we want to use our own PPA or archive for supplying apt dependencies. The PPA can still be added via the install_sources option.

To post a comment you must log in.
Revision history for this message
Stuart Bishop (stub) wrote :

Looks fine. test_wal_e_swift_logshipping will also need adjusting, but I can do that.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/reactive/postgresql/wal_e.py b/reactive/postgresql/wal_e.py
2index 231f645..d3b9bdc 100644
3--- a/reactive/postgresql/wal_e.py
4+++ b/reactive/postgresql/wal_e.py
5@@ -41,10 +41,10 @@ def main():
6 @when('postgresql.wal_e.enabled')
7 @when_not('apt.installed.wal_e')
8 def install():
9- # WAL-E is currently only available from a PPA. This charm and this
10- # PPA are maintained by the same person.
11- hookenv.log('Adding ppa:stub/pgcharm for wal-e packages')
12- apt.add_source('ppa:stub/pgcharm')
13+ # WAL-E is currently only available from a PPA, ppa:stub/pgcharm.
14+ # This charm and this PPA are maintained by the same person.
15+ # Ensure that this PPA or another containing the 'wal-e' package
16+ # is included as an install_source.
17 apt.queue_install(['daemontools', 'wal-e'])
18
19

Subscribers

People subscribed via source and target branches

to all changes: