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.

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

Refactor firewall policy module #1576

Merged
merged 14 commits into from
Aug 8, 2023
Merged

Refactor firewall policy module #1576

merged 14 commits into from
Aug 8, 2023

Conversation

ludoo
Copy link
Collaborator

@ludoo ludoo commented Aug 7, 2023

This implements the first part of the plan described in #1570 by refactoring the module so it manages all three firewall policy types, optional attachments, and a rule factory.

The implementation of rules is a bit convoluted to work around Terraform's type checks / conditionals making a mess of identically typed structures.

Leaving it as draft so we can look at it and decide if this is the way forward.

@ludoo ludoo requested review from juliocc and wiktorn August 7, 2023 17:14
@ludoo ludoo marked this pull request as ready for review August 8, 2023 15:13
@ludoo ludoo enabled auto-merge (squash) August 8, 2023 15:13
@ludoo ludoo disabled auto-merge August 8, 2023 15:14
@juliocc juliocc added the incompatible change Pull request that breaks compatibility with previous version label Aug 8, 2023
@ludoo ludoo enabled auto-merge (squash) August 8, 2023 16:42
@ludoo ludoo merged commit 80ada0e into master Aug 8, 2023
@ludoo ludoo deleted the ludo/issue-1570 branch August 8, 2023 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incompatible change Pull request that breaks compatibility with previous version on:modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants