-
Notifications
You must be signed in to change notification settings - Fork 9.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(region): support eu-west-3 region #2707
Conversation
looks like https://docs.aws.amazon.com/general/latest/gr/rande.html has a bit more info but still not everything needed |
Does this depend on PR in this repo to bump Related PR in |
From experimenting in the Paris region, it looks like the ELB Account ID is |
More interestingly, Redshift audit logging (which is the reason for requiring this AFAIK) is broken in eu-west-3. When I configure audit logging, the following happens:
|
It would be good if someone with support access and a vested interest in this PR could follow up with Amazon with regards to Redshift - that is the last blocker for this PR as far as I can see. |
Redshift service account ID has been released : http://docs.aws.amazon.com/redshift/latest/mgmt/db-auditing.html
|
AWS doc not updated
AWS doc not updated
@mvisonneau Great news! I'll look at getting this merged. |
@mvisonneau I'm going to pull this down locally to take it over the line. |
This has been merged in 132d10b - thanks @VEBERArnaud and @mvisonneau! Fixes #2735, |
Great news, how should I proceed to obtain the patch ? impatient to test it and hydrate our project.. |
This has been released in terraform-provider-aws version 1.7.0. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Description
add support for
eu-west-3 (Paris)
regionWIP
some ids are still missing from aws documentation, will update this PR when available
Depends
aws-sdk-go
to v1.12.49aws-sdk-go
to v1.12.50Resolve
resolves #2735