Switch naming in cr creation heade

Author: maciejriedl

pkg/firewalls/firewalls_l7_cr.go

@@ -84,7 +84,7 @@ func (fr *FirewallCR) Sync(nodeNames, additionalPorts, additionalRanges []string
 	ranges.Insert(additionalRanges...)
 
 	fr.logger.V(3).Info("Firewall CR is enabled.")
-	expectedFirewallCR, err := NewFirewallCR(name, ports.List(), ranges.UnsortedList(), []string{}, !fr.dryRun)
+	expectedFirewallCR, err := NewFirewallCR(name, ports.List(), ranges.UnsortedList(), []string{}, fr.dryRun)
 	if err != nil {
 		return err
 	}
@@ -141,14 +141,14 @@ func deleteFirewallCR(client firewallclient.Interface, name string, logger klog.
 }
 
 // NewFirewallCR constructs the firewall CR from name, ports and ranges
-func NewFirewallCR(name string, ports, srcRanges, dstRanges []string, enforced bool) (*gcpfirewallv1.GCPFirewall, error) {
+func NewFirewallCR(name string, ports, srcRanges, dstRanges []string, dryRun bool) (*gcpfirewallv1.GCPFirewall, error) {
 	firewallCR := &gcpfirewallv1.GCPFirewall{
 		ObjectMeta: metav1.ObjectMeta{
 			Name: name,
 		},
 		Spec: gcpfirewallv1.GCPFirewallSpec{
 			Action:   gcpfirewallv1.ActionAllow,
-			Disabled: !enforced,
+			Disabled: dryRun,
 		},
 	}
 	var protocolPorts []gcpfirewallv1.ProtocolPort