Skip to content

Commit

Permalink
Merge pull request #1148 from JacobTanenbaum/release-4.10-BZ2092895
Browse files Browse the repository at this point in the history
[release-4.10] Bug 2092895: update all egress ACLs' direction to "from-lport"
  • Loading branch information
openshift-merge-robot committed Feb 9, 2023
2 parents ff7233c + 60dc217 commit 48a174b
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions go-controller/pkg/ovn/policy.go
Original file line number Diff line number Diff line change
Expand Up @@ -1484,8 +1484,7 @@ func (oc *Controller) destroyNetworkPolicy(np *networkPolicy, lastPolicy bool) e
return fmt.Errorf("error cannot sync NetworkPolicy Egress obj: %v", err)
}
allEgressACLs = append(allEgressACLs, egressACLs...)
// if the first egress ACL is correct they should all be correct and not need to update
if allEgressACLs != nil && allEgressACLs[0].Direction != nbdb.ACLDirectionFromLport {
if len(allEgressACLs) != 0 {
// TODO(jtanenba) make all the libovsdbops.ACL commands deal with pointers to ACLs
var egressACLsPTR []*nbdb.ACL
for _, acl := range allEgressACLs {
Expand Down

0 comments on commit 48a174b

Please sign in to comment.