Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add support for local ranges to net-vpc-firewall #1303

Closed
juliocc opened this issue Apr 4, 2023 · 4 comments · Fixed by #1320
Closed

Add support for local ranges to net-vpc-firewall #1303

juliocc opened this issue Apr 4, 2023 · 4 comments · Fixed by #1320
Assignees
@ajlopezn

Comments

@juliocc
Copy link
Collaborator

juliocc commented Apr 4, 2023

Firewall rules now support source and destination ranges, provider support is available since version 4.49.

Extend the firewall module to support both fields in ingress and egress rules.
@ludoo
Copy link
Collaborator

ludoo commented Apr 4, 2023

THis is already implemented in the new firewall policy module.

@ludoo
Copy link
Collaborator

ludoo commented Apr 4, 2023

Ah, you're saying plain old firewall rules also support this?

@juliocc
Copy link
Collaborator Author

juliocc commented Apr 4, 2023

Ah, you're saying plain old firewall rules also support this?

Yes! thanks to @rosmo: GoogleCloudPlatform/magic-modules#6931

@ajlopezn
Copy link
Collaborator

I am on it

@ajlopezn ajlopezn mentioned this issue Apr 11, 2023
Merged
@ajlopezn ajlopezn linked a pull request Apr 11, 2023 that will close this issue
issue #1303: net-vpc-firewall module supporting source and destination ranges #1320
Merged
ajlopezn added a commit that referenced this issue Apr 12, 2023
@ajlopezn
Merge pull request #1320 from ajlopezn/ajln-firewall
a504738 
issue #1303: net-vpc-firewall module supporting source and destination ranges
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ajlopezn ajlopezn

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@ludoo @juliocc @ajlopezn