Fix static lint check

abhishek-pandey-1 · abhishek-pandey-1 · commit 82a316f71aad · 2026-02-25T10:30:11.000Z
Signed-off-by: Abhishek Pandey &lt;abhpandey@microsoft.com&gt;
diff --git a/pkg/ovs/ovn-nb-acl.go b/pkg/ovs/ovn-nb-acl.go
@@ -1133,11 +1133,14 @@ func (c *OVNNbClient) newSgRuleACL(sgName, direction string, rule kubeovnv1.Secu
 		}
 	}
 
-	action := ovnnb.ACLActionDrop
-	if rule.Policy == kubeovnv1.SgPolicyAllow {
+	var action string
+	switch rule.Policy {
+	case kubeovnv1.SgPolicyAllow:
 		action = ovnnb.ACLActionAllowRelated
-	} else if rule.Policy == kubeovnv1.SgPolicyPass {
+	case kubeovnv1.SgPolicyPass:
 		action = ovnnb.ACLActionPass
+	default:
+		action = ovnnb.ACLActionDrop
 	}
 
 	highestPriority, _ := strconv.Atoi(util.SecurityGroupHighestPriority)

Original file line number	Diff line number	Diff line change
`@@ -1133,11 +1133,14 @@ func (c *OVNNbClient) newSgRuleACL(sgName, direction string, rule kubeovnv1.Secu`
`1133`	`1133`	`}`
`1134`	`1134`	`}`
`1135`	`1135`
`1136`		`- action := ovnnb.ACLActionDrop`
`1137`		`- if rule.Policy == kubeovnv1.SgPolicyAllow {`
	`1136`	`+ var action string`
	`1137`	`+ switch rule.Policy {`
	`1138`	`+ case kubeovnv1.SgPolicyAllow:`
`1138`	`1139`	`action = ovnnb.ACLActionAllowRelated`
`1139`		`- } else if rule.Policy == kubeovnv1.SgPolicyPass {`
	`1140`	`+ case kubeovnv1.SgPolicyPass:`
`1140`	`1141`	`action = ovnnb.ACLActionPass`
	`1142`	`+ default:`
	`1143`	`+ action = ovnnb.ACLActionDrop`
`1141`	`1144`	`}`
`1142`	`1145`
`1143`	`1146`	`highestPriority, _ := strconv.Atoi(util.SecurityGroupHighestPriority)`