Uploaded image for project: 'Fast Datapath Product'
  1. Fast Datapath Product
  2. FDP-2072

Upstream: ovn-nbctl --wait=sb returns too early

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Undefined Undefined
    • None
    • None
    • ovn24.03
    • None
    • 3
    • False
    • False
    • Hide

      Please mark each item below with ( / ) if completed or ( x ) if incomplete:
      ( ) Unit test or Integration test case are written and pass successfully


      ( ) The upstream pull request is merged upstream and pass CI

      Show
      Please mark each item below with ( / ) if completed or ( x ) if incomplete: ( ) Unit test or Integration test case are written and pass successfully ( ) The upstream pull request is merged upstream and pass CI
    • rhel-9
    • None
    • rhel-net-ovn

      This is tracking the upstream effort needed to deliver the solution to the bug described below.


      The ovn-nbctl --wait=sb command could under certain conditions return earlier than the actual update is processed. The issue was discovered during review of https://patchwork.ozlabs.org/project/ovn/patch/20250915131602.54821-1-lucas.vdias@luizalabs.com/ the test would fail without the fix because northd entered commit fail loop. The expected behavior would be for the sync call to block until the timeout runs out. However the sync call was returning 0 and the test failed later on database check.

       

      The root cause is that northd I-P wouldn't run, but there was still valid transaction for SB. The transaction commit would return UNCHANGED, however, UNCHANGED moves idl loop next_cfg to cur_cfg, which is used by the synchronization. The next northd loop would update the SB nb_cfg in a different transaction than the failing one. Resulting in sync returning early.


              ovnteam@redhat.com OVN Team
              nstbot NST Bot
              OVN QE OVN QE
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: