-
Notifications
You must be signed in to change notification settings - Fork 3
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
Patch 2 #8
Patch 2 #8
Conversation
description = "Resource Discovery Description" | ||
operating_regions = [ |
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.
Both description
and operation_regions
need to be variables
cidr_filter_pattern = "10.*" # Optional, can be customized | ||
address_family = "IPv4" # Optional, can be customized |
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.
Both cidr_filter_pattern
and address_family
need to be varibles
mycidrs = [for cidr in data.aws_vpc_ipam_pool_cidrs.this.ipam_pool_cidrs : cidr.cidr if cidr.state == "provisioned"] | ||
} | ||
|
||
resource "aws_ec2_managed_prefix_list" "this" { |
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.
What is the concept for this resource? Not sure I understand the use case being filled here, in the context of IPAM.
description = "List of operating regions for IPAM resource discovery." | ||
type = list(object({ | ||
region_name = string | ||
# Include other attributes if needed |
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.
These comments end up in the README generated by terraform-docs. Remove unnecessary comments from this block
No description provided.