Juju Charms Collection
block-storage-broker package

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

  1. Precise Pangolin (12.04)
  2. bsb-ec2-support
  3. Merge into trunk
Revision history for this message
Chad Smith (chad.smith) wrote :

>
> [4]: If you specify your api endpoint as a valid region, but not the one
> you are currently in, you can get some confusing error messages.
> Noteably, something like:
>
> 'availability_zone'
>
> Then an error exit. It would be nice if it were more clear about what
> was failing. (this is not a blocker for this branch, maybe a follow-on
> MP).

[4] for this branch I've added a quick check if BSB is unable to find instance information for instance to which it is related. If no instance data is found, we know our charm config must be talking to a different region than the one our instances are running in so we exit(1) and log the following:

Could not create volume for instance i-77459754. No instance details discovered by euca-describe-instances. Maybe the charm configured endpoint https://ec2.us-west-1.amazonaws.com/ is not valid for this region.

« Back to merge proposal