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

Jerry, thanks for the updates and pulling in that volume-map and volume-label branch. I'll remove that MP now.

I've been deploying this now with the following bundle and I'm able to juju add-unit postgresql, juju remove-relation postgresql storage, juju add-relation postgresql storage and see the volumes remounted without hook issues. I'll debug a bit more and look into the race condition between data-relation and block-storage-relation hooks ordering.

cat postgres-storage.cfg
common:
    services:
        postgresql:
            branch: lp:~chad.smith/landscape/postgresql-storage-wip
            constraints: mem=2048
            options:
                extra-packages: python-apt postgresql-contrib postgresql-9.1-debversion
                max_connections: 500
        storage:
            branch: lp:~jseutter/charms/precise/storage/storage-with-broker-provider
            options:
                provider: block-storage-broker
                volume_size: 9
                volume_map: "{postgresql/0: 327fed5b-3ac7-45f7-988e-6a536cb32257}"
        block-storage-broker:
            branch: lp:~chad.smith/charms/precise/block-storage-broker/trunk
            options:
                key: <OS_USERNAME>
                endpoint: https://keystone.canonistack.canonical.com:443/v2.0/
                region: <OS_REGION_NAME>
                secret: <OS_PASSWORD>
                tenant: <OS_TENANT_NAME>

doit:
    inherits: common
    series: precise
    relations:
        - [postgresql, storage]
        - [storage, block-storage-broker]

$ juju-deployer -c postgres-storage.cfg doit