Added Controller build and initialization modules (and updated existing) #10
Conversation
|
|
||
| data "aws_region" "current" {} | ||
|
|
||
| locals { |
There was a problem hiding this comment.
Is it possible to use http Data Source to query the ami ID from https://s3-us-west-2.amazonaws.com/aviatrix-download/AMI_ID/ami_id.json? This would help avoid duplication
There was a problem hiding this comment.
I really like this idea. But, I can't figure out how to do this without the new jsondecode function from Terraform. We could write a function and put it here:
https://github.com/hashicorp/terraform/blob/master/config/interpolate_funcs.go (as suggested by this thread.
Let me know if I've missed another way of doing this.
| { | ||
| "Effect": "Allow", | ||
| "Action": [ | ||
| "elasticloadbalancing:RegisterInstancesWithLoadBalancer", |
There was a problem hiding this comment.
Same for the IAM policies. If we could query the latest IAM policy that would be ideal
There was a problem hiding this comment.
are those policies in a file somewhere? if they are in their own file then the http data source will work. if we need to parse the file, it seems like it won't work (unless I've missed something).
…ole arguments; general cleanup
aviatrix-controller-iam-roles - updated to latest version of policies
aviatrix-controller-build - updates to use metered AMI by default and added map for all regions
aviatrix-controller-build-cft - new module to build a controller using the CFT
aviatrix-controller-initialize - new module to initialize the Controller