Fix reconciliation triggers. - Githubissues

metal-stack / firewall-controller

A kubernetes controller running on bare-metal firewalls, creating nftables rules, configures suricata, collects network metrics

MIT License

49 stars 4 forks source link

Fix reconciliation triggers. #148

Closed Gerrit91 closed 1 year ago

Gerrit91 commented 1 year ago

Due to #146 it seems like the migration to v2 broke the conditions at when to reconcile some resources.

This PR attempts to fix the logic and make it a bit easier to understand:

The CWNP controller should run every interval through ticker logic
The CWNP controller should run on service resource changes
The firewall controller should run on firewall resource events
A new firewall monitor controller updates the firewall monitor in the shoot regularly