PaloAltoNetworks / terraform-aws-vmseries-modules

Terraform Reusable Modules for VM-Series on AWS
https://registry.terraform.io/modules/PaloAltoNetworks/vmseries-modules/aws
MIT License
40 stars 49 forks source link

Manage prefix-list for security-group #321

Closed David7660 closed 1 year ago

David7660 commented 1 year ago

Is your feature request related to a problem?

I'm frustrated to not be able to use my managed prefix-list as input for my security groups attached to my panorama or vmseries.

Describe the solution you'd like

I would like to be able to choose wether to put cidr_block or prefix-list as input on my security groups for my panorama or vmseries.

Describe alternatives you've considered.

For the moment I modified the module VPC on the main.tf on line 183-184 because I need it only for ingress for the moment cidr_blocks = try(ingress.value.cidr_blocks, null) prefix_list_ids = try(ingress.value.prefix_list, null)

Additional context

This issue affect the management of my repository on my Gitlab because I cannot mirror this repo due to this lacking feature as it is a requirement fr my security strategy to put prefix-list in security-groups

welcome-to-palo-alto-networks[bot] commented 1 year ago

:tada: Thanks for opening your first issue here! Welcome to the community!

sebastianczech commented 1 year ago

PR #325 is merged so I close the issue.