Bug 2080876 - Fix incorrectly logged error when searching for ovnkube master pod
Summary: Fix incorrectly logged error when searching for ovnkube master pod
Keywords:
Status: RELEASE_PENDING
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Networking
Version: 4.11
Hardware: All
OS: All
unspecified
low
Target Milestone: ---
: 4.11.0
Assignee: Ben Bennett
QA Contact: Anurag saxena
URL:
Whiteboard:
Depends On:
Blocks: 2080882
TreeView+ depends on / blocked
 
Reported: 2022-05-02 09:18 UTC by Martin Kennelly
Modified: 2022-05-02 09:20 UTC (History)
0 users

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 2080882 (view as bug list)
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Martin Kennelly 2022-05-02 09:18:52 UTC
Description of problem:
Before registering OVN northd metrics, we check to see if ovnkube
master is co-located on the same node and repeatedly check if it
is present because ovnkube node may start before ovnkube master.
This repeated checking only causes one error mode to occur which
is a timeout error and this is incorrectly logged as an error
confusing the user.
The only failure path to occur within 'checkPodRunsOnGivenNode' can
be seen with increased log level.


To verify this:
1. Grep the logs of the ovnkube-node container that isn't on a control plane node for the Error message 'Timed out while checking if OVNKube Master Pod runs'. You should not get a result.
2. Grep the logs of the ovnkube-node container that isn't on a control plane node for the Info message 'Not registering OVN North Metrics because OVNKube Master Pod was not found running on this'. You should get a result.




Version-Release number of selected component (if applicable):


How reproducible:
Always

Steps to Reproduce:
1. Grep the logs of the ovnkube-node container that isn't on a control plane node for the Error message 'Timed out while checking if OVNKube Master Pod runs'. 
2.
3.

Actual results:


Expected results:
Not error message but an Info message

Additional info:


Note You need to log in before you can comment on or make changes to this bug.