Skip to content

EC2 support for AWS Seoul region - #684

Closed
encyphered wants to merge 3 commits into
apache:trunkfrom
encyphered:trunk
Closed

EC2 support for AWS Seoul region#684
encyphered wants to merge 3 commits into
apache:trunkfrom
encyphered:trunk

Conversation

@encyphered

Copy link
Copy Markdown
Contributor
  • Fixed signature version of AWS Seoul region to 4 due to that does not support
  • Added provider and driver for region.

@tonybaloney

Copy link
Copy Markdown
Contributor

Thanks @encyphered do you have instances deployed in Seoul? Amazons press release said a subset of instance sizes for EC2 was supported, is this correct?

@encyphered

Copy link
Copy Markdown
Contributor Author

@tonybaloney Yes, Amazon announced only T2, M4, C4, I2, D2, and R3 was supported. HS1 is not supported

@tonybaloney

Copy link
Copy Markdown
Contributor

thanks. LGTM +1

@asfgit asfgit closed this in b0ce8d8 Jan 27, 2016
@Kami

Kami commented Jan 27, 2016

Copy link
Copy Markdown
Member

Thanks for the PR.

Does the new region only support new signature version 4? I'm asking because right now, IIRC, we only support signature version 4 for HTTP GET methods in Libcloud which means we should use 2 if it's still supported (otherwise non GET operations won't work).

@tonybaloney

Copy link
Copy Markdown
Contributor

I added POST request support for v4 about a month ago 

Sent from Outlook Mobile

On Wed, Jan 27, 2016 at 1:55 AM -0800, "Tomaz Muraus" notifications@github.com wrote:

Thanks for the PR.

Does the new region only support new signature version 4? I'm asking because right now, IIRC, we only support signature version 4 for HTTP GET methods in Libcloud which means we should use 2 if it's still supported (otherwise non GET operations won't work).


Reply to this email directly or view it on GitHub.

@Kami

Kami commented Jan 27, 2016

Copy link
Copy Markdown
Member

@tonybaloney Ah, nice! :)

I totally missed that.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants