Juju Charms Collection
postgresql package

Merge lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions into lp://staging/charms/trusty/postgresql

  1. Trusty Tahr (14.04)
  2. bug-1415490-max_prepared_transactions
  3. Merge into trunk
Proposed by Stuart Bishop
Status: Merged
Merged at revision: 118
Proposed branch: lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions
Merge into: lp://staging/charms/trusty/postgresql
Diff against target: 665 lines (+105/-87)
7 files modified
Makefile (+3/-1)
config.yaml (+72/-64)
hooks/helpers.py (+2/-4)
hooks/hooks.py (+14/-15)
hooks/test_hooks.py (+0/-1)
templates/postgresql.conf.tmpl (+13/-0)
test.py (+1/-2)
To merge this branch: bzr merge lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions
Reviewer Review Type Date Requested Status
Barry Price Approve
Review Queue (community) automated testing Needs Fixing
charmers Pending
Review via email: mp+250611@code.staging.launchpad.net

Description of the change

Add some tunables needed for Launchpad and Landscape deployments.

Also remove a config item that has done nothing for ages, and remove beta
comments about the no-longer beta PostgreSQL 9.4.

To post a comment you must log in.
lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions updated
120. By Stuart Bishop

Fix unittests

Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-11056-results

review: Needs Fixing (automated testing)
lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions updated
121. By Stuart Bishop

delint

Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-11057-results

review: Needs Fixing (automated testing)
lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions updated
122. By Stuart Bishop

delint

Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-11059-results

review: Needs Fixing (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-11094-results

review: Needs Fixing (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

This items has failed automated testing! Results available here http://reports.vapour.ws/charm-tests/charm-bundle-test-11099-results

review: Needs Fixing (automated testing)
lp://staging/~stub/charms/trusty/postgresql/bug-1415490-max_prepared_transactions updated
123. By Stuart Bishop

Checkpoint tuning options

Revision history for this message
Barry Price (barryprice) wrote :

Looks good to me, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: