Skip to content
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

Add China arn compatibility. Use arn_prefix var for determinate which… #41

Closed
wants to merge 1 commit into from

Conversation

gloterman
Copy link

Add China arn compatibility. Use arn_prefix var for determinate which ARN to use

what

  • Adapt the AWS ARN prefix pour S3 bucket resource.
  • Add a 'arn_prefix' local var in main.tf, the value is set related of the AWS region var.
  • In the S3 bucket policy template, add the 'local.arn_prefix' var in prefix of resources.
  • the 'arn_prefix' default value is set to 'arn:aws"

why

  • Because for China region the ARN is 'arn:aws-cn', instead of 'arn:aws' for all other regions.
  • It would be best if this module is available for all AWS regions.
  • In my business case we deploy in many regions, included China.

references

  • no reference, business case encountered in China region deployments.

@ericamador
Copy link
Contributor

I believe #39 should now address this issue.

@osterman
Copy link
Member

thanks @ericamador for the contribution. Yes #39 will replace this one as a generalied way to handle ARNs for any situation.

#39

@osterman osterman closed this Mar 26, 2020
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