Merge pull request #260 from willthames/aws_frankfurt
Added better region handling and enabled eu-central-1, leaving centralizing the list on these modules to a subsequent patch
This commit is contained in:
commit
0ab5682b87
8 changed files with 13 additions and 11 deletions
|
@ -57,7 +57,8 @@ options:
|
||||||
required: false
|
required: false
|
||||||
default: us-east-1
|
default: us-east-1
|
||||||
choices: ["ap-northeast-1", "ap-southeast-1", "ap-southeast-2",
|
choices: ["ap-northeast-1", "ap-southeast-1", "ap-southeast-2",
|
||||||
"eu-west-1", "sa-east-1", "us-east-1", "us-west-1", "us-west-2", "us-gov-west-1"]
|
"eu-central-1", "eu-west-1", "sa-east-1", "us-east-1",
|
||||||
|
"us-west-1", "us-west-2", "us-gov-west-1"]
|
||||||
virt:
|
virt:
|
||||||
description: virutalization type
|
description: virutalization type
|
||||||
required: false
|
required: false
|
||||||
|
@ -89,11 +90,13 @@ SUPPORTED_DISTROS = ['ubuntu']
|
||||||
AWS_REGIONS = ['ap-northeast-1',
|
AWS_REGIONS = ['ap-northeast-1',
|
||||||
'ap-southeast-1',
|
'ap-southeast-1',
|
||||||
'ap-southeast-2',
|
'ap-southeast-2',
|
||||||
|
'eu-central-1',
|
||||||
'eu-west-1',
|
'eu-west-1',
|
||||||
'sa-east-1',
|
'sa-east-1',
|
||||||
'us-east-1',
|
'us-east-1',
|
||||||
'us-west-1',
|
'us-west-1',
|
||||||
'us-west-2']
|
'us-west-2',
|
||||||
|
"us-gov-west-1"]
|
||||||
|
|
||||||
|
|
||||||
def get_url(module, url):
|
def get_url(module, url):
|
||||||
|
|
|
@ -274,7 +274,7 @@ def create_autoscaling_group(connection, module):
|
||||||
region, ec2_url, aws_connect_params = get_aws_connection_info(module)
|
region, ec2_url, aws_connect_params = get_aws_connection_info(module)
|
||||||
try:
|
try:
|
||||||
ec2_connection = connect_to_aws(boto.ec2, region, **aws_connect_params)
|
ec2_connection = connect_to_aws(boto.ec2, region, **aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
module.fail_json(msg=str(e))
|
module.fail_json(msg=str(e))
|
||||||
elif vpc_zone_identifier:
|
elif vpc_zone_identifier:
|
||||||
vpc_zone_identifier = ','.join(vpc_zone_identifier)
|
vpc_zone_identifier = ','.join(vpc_zone_identifier)
|
||||||
|
|
|
@ -258,7 +258,7 @@ class ElbManager:
|
||||||
try:
|
try:
|
||||||
elb = connect_to_aws(boto.ec2.elb, self.region,
|
elb = connect_to_aws(boto.ec2.elb, self.region,
|
||||||
**self.aws_connect_params)
|
**self.aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
self.module.fail_json(msg=str(e))
|
self.module.fail_json(msg=str(e))
|
||||||
|
|
||||||
elbs = elb.get_all_load_balancers()
|
elbs = elb.get_all_load_balancers()
|
||||||
|
@ -278,7 +278,7 @@ class ElbManager:
|
||||||
try:
|
try:
|
||||||
ec2 = connect_to_aws(boto.ec2, self.region,
|
ec2 = connect_to_aws(boto.ec2, self.region,
|
||||||
**self.aws_connect_params)
|
**self.aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
self.module.fail_json(msg=str(e))
|
self.module.fail_json(msg=str(e))
|
||||||
return ec2.get_only_instances(instance_ids=[self.instance_id])[0]
|
return ec2.get_only_instances(instance_ids=[self.instance_id])[0]
|
||||||
|
|
||||||
|
|
|
@ -375,7 +375,7 @@ class ElbManager(object):
|
||||||
try:
|
try:
|
||||||
return connect_to_aws(boto.ec2.elb, self.region,
|
return connect_to_aws(boto.ec2.elb, self.region,
|
||||||
**self.aws_connect_params)
|
**self.aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
self.module.fail_json(msg=str(e))
|
self.module.fail_json(msg=str(e))
|
||||||
|
|
||||||
def _delete_elb(self):
|
def _delete_elb(self):
|
||||||
|
|
|
@ -63,6 +63,7 @@ class Ec2Metadata(object):
|
||||||
AWS_REGIONS = ('ap-northeast-1',
|
AWS_REGIONS = ('ap-northeast-1',
|
||||||
'ap-southeast-1',
|
'ap-southeast-1',
|
||||||
'ap-southeast-2',
|
'ap-southeast-2',
|
||||||
|
'eu-central-1',
|
||||||
'eu-west-1',
|
'eu-west-1',
|
||||||
'sa-east-1',
|
'sa-east-1',
|
||||||
'us-east-1',
|
'us-east-1',
|
||||||
|
|
|
@ -265,7 +265,7 @@ def main():
|
||||||
|
|
||||||
try:
|
try:
|
||||||
connection = connect_to_aws(boto.ec2.autoscale, region, **aws_connect_params)
|
connection = connect_to_aws(boto.ec2.autoscale, region, **aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
module.fail_json(msg=str(e))
|
module.fail_json(msg=str(e))
|
||||||
|
|
||||||
state = module.params.get('state')
|
state = module.params.get('state')
|
||||||
|
|
|
@ -271,7 +271,7 @@ def main():
|
||||||
region, ec2_url, aws_connect_params = get_aws_connection_info(module)
|
region, ec2_url, aws_connect_params = get_aws_connection_info(module)
|
||||||
try:
|
try:
|
||||||
connection = connect_to_aws(boto.ec2.cloudwatch, region, **aws_connect_params)
|
connection = connect_to_aws(boto.ec2.cloudwatch, region, **aws_connect_params)
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
module.fail_json(msg=str(e))
|
module.fail_json(msg=str(e))
|
||||||
|
|
||||||
if state == 'present':
|
if state == 'present':
|
||||||
|
|
|
@ -163,9 +163,7 @@ def main():
|
||||||
|
|
||||||
try:
|
try:
|
||||||
connection = connect_to_aws(boto.ec2.autoscale, region, **aws_connect_params)
|
connection = connect_to_aws(boto.ec2.autoscale, region, **aws_connect_params)
|
||||||
if not connection:
|
except (boto.exception.NoAuthHandlerFound, StandardError), e:
|
||||||
module.fail_json(msg="failed to connect to AWS for the given region: %s" % str(region))
|
|
||||||
except boto.exception.NoAuthHandlerFound, e:
|
|
||||||
module.fail_json(msg = str(e))
|
module.fail_json(msg = str(e))
|
||||||
|
|
||||||
if state == 'present':
|
if state == 'present':
|
||||||
|
|
Loading…
Reference in a new issue