[ovs-git] [ovn-org/ovn] 5adcd1: Fix OVN update issue when ovn-controller is update...

numansiddique noreply at github.com
Mon Nov 23 10:19:01 UTC 2020


  Branch: refs/heads/branch-20.09
  Home:   https://github.com/ovn-org/ovn
  Commit: 5adcd1409a2912b13a91015fd90e28dfd41ea4c0
      https://github.com/ovn-org/ovn/commit/5adcd1409a2912b13a91015fd90e28dfd41ea4c0
  Author: Numan Siddique <numans at ovn.org>
  Date:   2020-11-23 (Mon, 23 Nov 2020)

  Changed paths:
    M include/ovn/actions.h
    M lib/actions.c
    M tests/ovn.at
    M utilities/ovn-trace.c

  Log Message:
  -----------
  Fix OVN update issue when ovn-controller is updated first from 20.06 to 20.09.

The commit in the Fixes tag changed the ct_commit signature from
ct_commit(..) to ct_commit{ ..}.  When ovn-controllers are updated
to a vesion which has this change and if ovn-northd is still not
udated to this version, then the logical flow with the action
ct_commit(..) will be rejected by ovn-controllers resulting in
datapath disruptions.  OVN recommends ovn-controller updates before
ovn-northd, but it is broken now.  This patch fixes this issue by
adding back the support for the old ct_commit(..).  We now support
both the versions of ct_commit.

Fixes: 6cfb44a76c61("Used nested actions in ct_commit)
CC: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>
Acked-by: Flavio Fernandes <flavio at flaviof.com>
Signed-off-by: Numan Siddique <numans at ovn.org>

(cherry-picked from master commit 4799f73fd6edbab01f33ffc181b12f24f55b2ac7)




More information about the git mailing list