Merge pull request #2 from sam-h-bean/mrbean/change-provider

Mrbean/change provider

README.md

@@ -13,8 +13,8 @@ Terraform Module for managing s3 bucket cross-account cross-region replication.
 
 - Terraform 0.11 module provider inheritance block:
 
-- `aws.source` - AWS provider alias for source account
-- `aws.dest`   - AWS provider alias for destination account
+- `aws.source_of_replication` - AWS provider alias for source account
+- `aws.destination_of_replication`   - AWS provider alias for destination account
 
 #### Optional
 
@@ -47,8 +47,8 @@ module "s3-cross-account-replication" {
   replication_name   = "my-replication-name"
 
   providers {
-    "aws.source" = "aws.source"
-    "aws.dest"   = "aws.dest"
+    "aws.source_of_replication" = "source_of_replication"
+    "aws.destination_of_replication"   = "aws.destination_of_replication"
   }
 }
 
@@ -81,7 +81,11 @@ Authors
 Changelog
 =========
 
-1.0.0 - Initial release.
+1.0.0 - Initial release
+
+2.0.0 - TF 0.12.X
+
+3.0.0 - Rename povider
 
 License
 =======

iam-source.tf

@@ -37,25 +37,25 @@ data "aws_iam_policy_document" "source_write" {
 }
 
 resource "aws_iam_policy" "source_write" {
-  provider    = aws.source
+  provider    = aws.source_of_replication
   name_prefix = "${local.replication_name}-source-write-"
   policy      = data.aws_iam_policy_document.source_write.json
 }
 
 resource "aws_iam_user" "source_write" {
-  provider      = aws.source
+  provider      = aws.source_of_replication
   name          = "${local.replication_name}-source-write-user"
   force_destroy = true
 }
 
 resource "aws_iam_user_policy_attachment" "source_write" {
-  provider   = aws.source
+  provider   = aws.source_of_replication
   user       = aws_iam_user.source_write.name
   policy_arn = aws_iam_policy.source_write.arn
 }
 
 resource "aws_iam_access_key" "source_write" {
-  provider = aws.source
+  provider = aws.source_of_replication
   user     = aws_iam_user.source_write.name
 }
 

main.tf

@@ -1,18 +1,18 @@
 # MAIN
 
 provider "aws" {
-  alias = "source"
+  alias = "source_of_replication"
 }
 
 provider "aws" {
-  alias = "dest"
+  alias = "destination_of_replication"
 }
 
 data "aws_caller_identity" "source" {
-  provider = aws.source
+  provider = aws.source_of_replication
 }
 
 data "aws_caller_identity" "dest" {
-  provider = aws.dest
+  provider = aws.destination_of_replication
 }
 

s3-dest.tf

@@ -42,7 +42,7 @@ data "aws_iam_policy_document" "dest_bucket_policy" {
 
 resource "aws_s3_bucket" "dest" {
   count    = var.create_dest_bucket == "true" ? 1 : 0
-  provider = aws.dest
+  provider = aws.destination_of_replication
   bucket   = var.dest_bucket_name
   region   = var.dest_region
   policy   = data.aws_iam_policy_document.dest_bucket_policy.json

s3-source.tf

@@ -74,27 +74,27 @@ data "aws_iam_policy_document" "source_replication_policy" {
 }
 
 resource "aws_iam_role" "source_replication" {
-  provider           = aws.source
+  provider           = aws.source_of_replication
   name               = "${local.replication_name}-replication-role"
   assume_role_policy = data.aws_iam_policy_document.source_replication_role.json
 }
 
 resource "aws_iam_policy" "source_replication" {
-  provider = aws.source
+  provider = aws.source_of_replication
   name     = "${local.replication_name}-replication-policy"
   policy   = data.aws_iam_policy_document.source_replication_policy.json
 }
 
 resource "aws_iam_role_policy_attachment" "source_replication" {
-  provider   = aws.source
+  provider   = aws.source_of_replication
   role       = aws_iam_role.source_replication.name
   policy_arn = aws_iam_policy.source_replication.arn
 }
 
 # S3 source bucket
 
 resource "aws_s3_bucket" "source" {
-  provider = aws.source
+  provider = aws.source_of_replication
   bucket   = var.source_bucket_name
   region   = var.source_region