ovn-kubernetes / ovn-kubernetes / 20884660012

54%

master: 54%

Build Type

Pull #5623

github

Committed by trozet

Commit Message

Critical Dynamic UDN NAD controller fixes

There were two issues:
1. During a marked for deletion if the network controller did not exist,
   the primary NAD cache would not be updated, but the nads cache of nad
   controller would, leading to a panic. Updated the code to update the
   primaryNADs cache regardless of if there is a network controller or
   not.

2. During a marked for deletion, the primaryNAD cache would have its
   entry deleted. This would then cause a chicken and egg problem where
   egressIP going active on a node, relies on the primaryNAD cache, but
   the cache is empty, and NAD controller wont add the primaryNAD back
   because it is filtered by the trackers. Fix is to not modify cache
   based on dynamic UDN. The caches should always be driven by what is
   in kapi.

Signed-off-by: Tim Rozet <trozet@nvidia.com>

Pull Request Pull Request #5623: Dynamic UDN Allocation Feature Implementation

Run Details

653 of 1162 new or added lines in 21 files covered. (56.2%)

41 existing lines in 11 files now uncovered.

41689 of 76692 relevant lines covered (54.36%)

421.03 hits per line