Merge lp:~verterok/landscape-charm/support-postgresql-charm-v2-protocol into lp:~landscape/landscape-charm/trunk
Proposed by
Guillermo Gonzalez
Status: | Merged |
---|---|
Approved by: | Guillermo Gonzalez |
Approved revision: | 402 |
Merged at revision: | 399 |
Proposed branch: | lp:~verterok/landscape-charm/support-postgresql-charm-v2-protocol |
Merge into: | lp:~landscape/landscape-charm/trunk |
Diff against target: |
296 lines (+78/-61) 9 files modified
lib/apt.py (+1/-1) lib/callbacks/scripts.py (+1/-1) lib/callbacks/tests/test_scripts.py (+11/-0) lib/relations/postgresql.py (+30/-35) lib/relations/tests/test_postgresql.py (+18/-17) lib/services.py (+2/-1) lib/tests/sample.py (+4/-3) lib/tests/stubs.py (+1/-1) lib/tests/test_services.py (+10/-2) |
To merge this branch: | bzr merge lp:~verterok/landscape-charm/support-postgresql-charm-v2-protocol |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Simon Poirier (community) | Approve | ||
🤖 Landscape Builder | test results | Approve | |
Review via email: mp+370150@code.launchpad.net |
This proposal supersedes a proposal from 2019-07-10.
Commit message
use postgresql-charm v2 protocol, manually parse the dsn as the available psycopg2 (2.6.x) doesn't provide parse_dsn (added in 2.7.x)
Description of the change
use postgresql-charm v2 protocol.
As we are using system packages for python deps, manually parse the dsn as the available psycopg2 (2.6.x) doesn't provide parse_dsn (added in 2.7.x).
This change is to be able to replace postgresql unit in the juju deploy with a external-services charm which acts as a proxy to connect to metal/external deployments of postgresql.
To post a comment you must log in.
Command: make ci-test /ci.lscape. net/job/ latch-test- xenial/ 3974/
Result: Fail
Revno: 400
Branch: lp:~verterok/landscape-charm/support-postgresql-charm-v2-protocol
Jenkins: https:/