terraform-google-modules / terraform-google-network

Sets up a new VPC network on Google Cloud
https://registry.terraform.io/modules/terraform-google-modules/network/google
Apache License 2.0
410 stars 1.23k forks source link

Dedicated firewall rules module + adding to top level module #199

Closed umairidris closed 1 year ago

umairidris commented 4 years ago

Currently the only CFT module to create firewall rules seems to be https://github.com/terraform-google-modules/terraform-google-network/tree/master/modules/fabric-net-firewall.

This seems to create a few default firewall rules and also needs to be a separate module call.

There is functionality to create custom rules. Would you be willing to accept a PR that pulls the custom rules logic out to its own submodule and adds it under the top level network module? Thus it would allow a user to create a network with custom firewall rules from one call.

morgante commented 4 years ago

Yeah I think we need to refactor the firewall rules module a bit. Here's what I'd like to see:

  1. Split the core rules logic into its own submodule
  2. Put the default rules in a submodule (which calls the core rules module)
  3. Add a reference to (1) from the root module.
github-actions[bot] commented 3 years ago

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

morgante commented 3 years ago

Still needs to happen.

imrannayer commented 1 year ago

I think sub-module firewall-rules covers it.