-
Notifications
You must be signed in to change notification settings - Fork 163
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
aws: new resources #87
Conversation
09cbd9d
to
cb107ac
Compare
@@ -34,6 +34,16 @@ var ( | |||
// Returned values are commented in the interface doc comment block. | |||
`, | |||
}, | |||
Function{ | |||
Entity: "VpcPeeringConnections", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it maybe be named VPC..
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took the format from https://github.com/cycloidio/terracognita/blob/master/aws/cmd/functions.go#L29
Should we change both to VPC ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok nothing, this is the one used on their side, it's ok :).
aws/resources.go
Outdated
} | ||
} | ||
|
||
//TODO: This resource it's not Importable yet (https://www.terraform.io/docs/providers/aws/r/lb_listener_certificate.html) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add an issue and link it to this so we know/remember this if at somepoint it's importable.
If not, maybe we do not need this function as it has no use hehe.
PR updated |
yes I'm reverting VPCPeeringConnections to VpcPeeringConnections but I failed somewhere :) |
aws_alb_listener aws_alb_target_group aws_alb_listener_rule aws_alb_listener_certificate (not importable yet)
Generate aws/reader/reader.go aws/resourcetype_enumer.go
Implement new AWS resources