Bug 2108317

Summary:	Fix two issues in hybrid overlay
Product:	OpenShift Container Platform	Reporter:	Dan Williams <dcbw>
Component:	Networking	Assignee:	Dan Williams <dcbw>
Networking sub component:	ovn-kubernetes	QA Contact:	Anurag saxena <anusaxen>
Status:	CLOSED ERRATA	Docs Contact:
Severity:	high
Priority:	high	CC:	rravaiol
Version:	4.12
Target Milestone:	---
Target Release:	4.12.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	2109642 (view as bug list)		Environment:
Last Closed:	2023-01-17 19:53:01 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	2109642

Description Dan Williams 2022-07-18 20:31:13 UTC

Cherry-pick of https://github.com/ovn-org/ovn-kubernetes/pull/3063 from upstream.

A missing comma before an "action=" meant that those OVS flows weren't getting added due to a parsing error.

Second, the `k8s.ovn.org/node-gateway-router-lrp-ifaddr` annotation added by https://github.com/ovn-org/ovn-kubernetes/pull/2862 was only evaluated on node add, but we can't always rely on that. It might be added slightly later in an Update event.

Comment 1 Dan Williams 2022-07-18 20:32:19 UTC

The missing comma is a real problem for Windows hybrid clusters.

Comment 3 Dan Williams 2022-07-22 13:18:33 UTC

Downstream 4.12 merge PR merged which contains this fix.

Comment 7 errata-xmlrpc 2023-01-17 19:53:01 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.12.0 bug fix and security update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:7399