Merge pull request 'Properly check for firewallGroup in same namespace' (#33) from feature/properly-handle-firewallgroups-in-same-namespace into main
All checks were successful
Publish / build (push) Successful in 2m27s
All checks were successful
Publish / build (push) Successful in 2m27s
Reviewed-on: #33
This commit is contained in:
@@ -287,7 +287,7 @@ func (r *FirewallPolicyReconciler) Reconcile(ctx context.Context, req ctrl.Reque
|
|||||||
// This will be used when running through all firewall groups and servics known, to see if a rule should be added.
|
// This will be used when running through all firewall groups and servics known, to see if a rule should be added.
|
||||||
|
|
||||||
for _, dest_group := range firewallPolicy.Spec.Destination.FirewallGroups {
|
for _, dest_group := range firewallPolicy.Spec.Destination.FirewallGroups {
|
||||||
namespace := defaultNs
|
namespace := firewallPolicy.Namespace
|
||||||
if len(dest_group.Namespace) > 0 {
|
if len(dest_group.Namespace) > 0 {
|
||||||
namespace = dest_group.Namespace
|
namespace = dest_group.Namespace
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user