Code review comment for lp://staging/~jseutter/charms/precise/storage/storage-with-broker-provider

Revision history for this message
David Britton (dpb) wrote :

I'd like to re-review, but there seems to be some rework necessary here. I'd suggest for testing:

1) deploy
2) add-unit postgres
3) add-unit postgres
4) break relation between postgres and storage, add back
5) destroy-services
6) make sure everything comes down without error

In between each step, make sure the system looks right. That the volume is mounted, that data is being stored there for postgres. This can be seen with the following checks as user postgres:

* ls -ld /var/lib/postgresql/9.1/main
* ls -l /mnt/takeitlikeanova
* psql # CREATE DATABASE foo -- do this once on the main node
* psql -d foo # make sure with each new node, you can see the data copied in.
* pg_lsclusters

review: Needs Fixing

« Back to merge proposal