Support multiple RAM principals

[alexjurkiewicz]

Introduce new variable var.ram_principals, and deprecate var.ram_principal. Deprecating rather than removing the old variable is a lot more complex, code-wise, but the complexity is contained and should ease the upgrade path for existing customers.

Fixes #14

[alexjurkiewicz]

Thanks for the early review. I found a few other bugs and I'm testing internally. I will convert this PR to draft so you don't waste time 🙏

[alexjurkiewicz]

Looks like it's working to me! I can specify two external AWS account IDs and two aws_ram_principal_association are created.

[aknysh]

/test all

[aknysh]

@alexjurkiewicz thanks again

please see the terratest error

Error: Invalid for_each argument
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66: 
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66:   on ../../ram.tf line 34, in resource "aws_ram_principal_association" "default":
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66:   34:   for_each           = local.ram_principals
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66: 
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66: The given "for_each" argument value is unsuitable: the "for_each" argument
TestExamplesComplete 2022-03-15T15:12:53Z logger.go:66: must be a map, or set of strings, and you have provided a value of type tuple.

[alexjurkiewicz]

I've updated the PR

[Gowiem]

/test all

[aknysh]

/test all

[aknysh]

/test all

[alexjurkiewicz]

thank you for all the help Andriy 😊