Merge lp://qastaging/~ahasenack/charms/precise/postgresql/postgresql-avoid-cname into lp://qastaging/charms/postgresql
Proposed by
Andreas Hasenack
| Status: | Merged |
|---|---|
| Merged at revision: | 75 |
| Proposed branch: | lp://qastaging/~ahasenack/charms/precise/postgresql/postgresql-avoid-cname |
| Merge into: | lp://qastaging/charms/postgresql |
| Diff against target: |
20 lines (+10/-0) 1 file modified
hooks/hooks.py (+10/-0) |
| To merge this branch: | bzr merge lp://qastaging/~ahasenack/charms/precise/postgresql/postgresql-avoid-cname |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Stuart Bishop (community) | Approve | ||
|
Review via email:
|
|||
Description of the change
Temporary hack for the postgresql charm to avoid using CNAME records for access control in pg_hba.conf (see bug #1250435).
The downside is that there can be repeated calls for "dig", which is a network query in the end. It could use some caching, but since the MAAS bug is bound to be fixed soon, my hope is that this hack can also be removed soon.
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
Looks fine for a temporary work around. dig(1) appears to be installed by default, so no need for extra packages to be added.