Code review comment for lp:~chad.smith/charms/precise/block-storage-broker/bsb-ec2-support

Hey Chad, got a failure while deploying:

fcorrea@beast:~/src/landscape/landscape-charm/config: juju-deployer -c postgres-storage-ec2.cfg doit -e fcorrea-amazon
2014-03-20 17:38:53 Starting deployment of doit
2014-03-20 17:39:13 Deploying services...
2014-03-20 17:39:16 Deploying service block-storage-broker-ec2 using local:precise/block-storage-broker
2014-03-20 17:39:32 Deploying service postgresql using local:precise/postgresql
2014-03-20 17:39:42 Deploying service storage using local:precise/storage
2014-03-20 17:40:06 Config specifies num units for subordinate: storage
2014-03-20 17:43:33 Adding relations...
2014-03-20 17:43:34 Adding relation postgresql <-> storage
Traceback (most recent call last):
  File "/usr/bin/juju-deployer", line 9, in <module>
    load_entry_point('juju-deployer==0.3.4', 'console_scripts', 'juju-deployer')()
  File "/usr/lib/python2.7/dist-packages/deployer/cli.py", line 127, in main
    run()
  File "/usr/lib/python2.7/dist-packages/deployer/cli.py", line 225, in run
    importer.Importer(env, deployment, options).run()
  File "/usr/lib/python2.7/dist-packages/deployer/action/importer.py", line 198, in run
    rels_created = self.add_relations()
  File "/usr/lib/python2.7/dist-packages/deployer/action/importer.py", line 128, in add_relations
    self.env.add_relation(end_a, end_b)
  File "/usr/lib/python2.7/dist-packages/deployer/env/go.py", line 57, in add_relation
    return self.client.add_relation(endpoint_a, endpoint_b)
  File "/usr/lib/python2.7/dist-packages/jujuclient.py", line 561, in add_relation
    'Endpoints': [endpoint_a, endpoint_b]
  File "/usr/lib/python2.7/dist-packages/jujuclient.py", line 152, in _rpc
    raise EnvError(result)
jujuclient.EnvError: <Env Error - Details:
 { u'Error': u'no relations found', u'RequestId': 1, u'Response': { }}
 >

I've tested juju-deployer by deploying the landscape-charm and it was fine.

This is what I have in my postgres-storage-ec2.cfg

common:
    services:
        postgresql:
            branch: lp:~chad.smith/charms/precise/postgresql/postgresql-using-storage-subordinate
            constraints: mem=2048
            options:
                extra-packages: python-apt postgresql-contrib postgresql-9.1-debversion
                max_connections: 500
        storage:
            branch: lp:~chad.smith/charms/precise/storage/storage-ec2-support
            options:
                provider: block-storage-broker
                volume_size: 9
        block-storage-broker-ec2:
            branch: lp:~chad.smith/charms/precise/block-storage-broker/bsb-ec2-support
            options:
                provider: ec2
                key: <the same key I use to run euca-commands>
                endpoint: https://ec2.us-east-1.amazonaws.com
                secret: <the same secret I use to run euca-commands>

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

Please, lemme know if I can provide you with any extra info.