Add support for AWS Australia (ap-southeast-2). #1271

Merged
merged 2 commits into from Nov 14, 2012

2 participants

@aswoodward

No description provided.

@geemus geemus and 3 others commented on an outdated diff Nov 14, 2012
lib/fog/aws/requests/dns/change_resource_record_sets.rb
@@ -198,6 +198,7 @@ def self.elb_hosted_zone_mapping
@elb_hosted_zone_mapping ||= {
"ap-northeast-1" => "Z2YN17T5R711GT",
"ap-southeast-1" => "Z1WI8VXHPB1R38",
+ "ap-southeast-2" => "Z1WI8VXHPB1R38", # TODO Needs updated value
@geemus
fog member
geemus added a line comment Nov 14, 2012

Maybe if the value is incorrect we should just leave it out.

@bgentry - I think you got these values via our TAM, any chance we can get this one also?

@dpiddy
fog member
dpiddy added a line comment Nov 14, 2012

Creating an ELB in this region should give you the hosted zone ID.

@bgentry
bgentry added a line comment Nov 14, 2012
@bgentry
bgentry added a line comment Nov 14, 2012

My trick worked. Took a us-east-1 ELB hostname, pasted into the R53 Alias Target box on the web console, edited the region to ap-southeast-2, and it showed me this:

Alias Hosted Zone ID: Z2999QAZ9SRTIC
@geemus
fog member
geemus added a line comment Nov 14, 2012

@bgentry - awesome, thanks!

@aswoodward
aswoodward added a line comment Nov 14, 2012

Indeed - @bgentry thanks! I'll add that in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@geemus geemus commented on the diff Nov 14, 2012
lib/fog/aws/models/compute/server.rb
@@ -60,6 +60,8 @@ def initialize(attributes={})
'ami-5e0fa45f'
when 'ap-southeast-1'
'ami-f092eca2'
+ when 'ap-southeast-2'
+ 'ami-fb8611c1' # Ubuntu 12.04 LTS 64bit (EBS)
@geemus
fog member
geemus added a line comment Nov 14, 2012

It slipped my mind when we discussed this, but I think the other ones may be instance store 10.04 instead of EBS. Not sure how much this matters but we should perhaps discuss. Thoughts?

@aswoodward
aswoodward added a line comment Nov 14, 2012

@geemus: Line 58's comment implies that the other ones are also EBS:

case attributes[:connection].instance_variable_get(:@region) # Ubuntu 10.04 LTS 64bit (EBS)
@geemus
fog member
geemus added a line comment Nov 14, 2012

Fair enough, probably misremembered. Fine to leave this as-is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@geemus
fog member

@aswoodward - awesome, thanks. Added a couple small points for discussion above.

@geemus geemus merged commit f4ae7a1 into fog:master Nov 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment