Juju Charms Collection
postgresql package

Merge lp://qastaging/~stub/charms/precise/postgresql/bug-1329816-backup-log-spurious-alert into lp://qastaging/charms/postgresql

Proposed by Stuart Bishop on 2014-06-16

Status:	Merged
Merged at revision:	101
Proposed branch:	lp://qastaging/~stub/charms/precise/postgresql/bug-1329816-backup-log-spurious-alert
Merge into:	lp://qastaging/charms/postgresql
Prerequisite:	lp://qastaging/~stub/charms/precise/postgresql/tests
Diff against target:	33 lines (+8/-1) 1 file modified hooks/hooks.py (+8/-1)
To merge this branch:	bzr merge lp://qastaging/~stub/charms/precise/postgresql/bug-1329816-backup-log-spurious-alert
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Michael Nelson (community)		2014-06-16	Approve on 2014-06-16
Review via email: mp+223210@code.qastaging.launchpad.net

The backup log file does not exist until the first backup runs, causing a spurious nagios alert on a freshly deployed service.

This branch ensures the log file is created on initial installation.

Revision history for this message

Michael Nelson (michael.nelson) wrote on 2014-06-16:

Excellent - thanks Stuart!

review: Approve

142. By Stuart Bishop on 2014-06-16: revert unnecessary whitespace change
143. By Stuart Bishop on 2014-06-27: Merged tests into bug-1329816-backup-log-spurious-alert.
144. By Stuart Bishop on 2014-06-27: Merged tests into bug-1329816-backup-log-spurious-alert.
145. By Stuart Bishop on 2014-06-30: Merged tests into bug-1329816-backup-log-spurious-alert.