[ovs-git] [openvswitch/ovs] 8e6934: conntrack: Fix the icmp conntrack new state.

Ilya Maximets noreply at github.com
Wed Jan 27 16:44:38 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 8e69349f2eb31a2ed6d9dfc4c1cad40259d84142
      https://github.com/openvswitch/ovs/commit/8e69349f2eb31a2ed6d9dfc4c1cad40259d84142
  Author: Tonghao Zhang <xiangxia.m.yue at gmail.com>
  Date:   2021-01-26 (Tue, 26 Jan 2021)

  Changed paths:
    M lib/conntrack-icmp.c
    M tests/system-common-macros.at
    M tests/system-traffic.at

  Log Message:
  -----------
  conntrack: Fix the icmp conntrack new state.

The same icmp packet may traverse conntrack module more than once.
Or same icmp packets traverse contranck module in orderly.

Don't change state to CS_ESTABLISHED before receiving reply or related packets.

Fixes: a867c010ee91 ("conntrack: Fix conntrack new state")
Signed-off-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
Acked-by: Yi-Hung Wei <yihung.wei at gmail.com>
Acked-by: Aaron Conole <aconole at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 82440ad72dd64698fec3d6c562c127f235269139
      https://github.com/openvswitch/ovs/commit/82440ad72dd64698fec3d6c562c127f235269139
  Author: Mao YingMing <maoyingming at baidu.com>
  Date:   2021-01-27 (Wed, 27 Jan 2021)

  Changed paths:
    M lib/dpctl.c

  Log Message:
  -----------
  dpctl: Fix dpctl process command parameter error.

Fix the following error:

  $ ovs-appctl dpctl/dump-conntrack -m -s system at ovs-system zone=0
  "dpctl/dump-conntrack" command takes at most 2 arguments
  ovs-appctl: ovs-vswitchd: server returned an error

  $ ovs-appctl dpctl/dump-flows -m --names system at ovs-system filter=in_port\(2\) type=ovs
  "dpctl/dump-flows" command takes at most 4 arguments
  ovs-appctl: ovs-vswitchd: server returned an error

Signed-off-by: Mao YingMing <maoyingming at baidu.com>
Acked-by: Mark Gray <mark.d.gray at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: c2e501c16b1287fd8357ec7db696524fb07c926c
      https://github.com/openvswitch/ovs/commit/c2e501c16b1287fd8357ec7db696524fb07c926c
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2021-01-27 (Wed, 27 Jan 2021)

  Changed paths:
    M AUTHORS.rst

  Log Message:
  -----------
  AUTHORS: Add Mao YingMing.

Additionally, re-sorted this part of the list.

Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 25eb72a26b18235207d1c7a614a2f30a725bfa77
      https://github.com/openvswitch/ovs/commit/25eb72a26b18235207d1c7a614a2f30a725bfa77
  Author: Flavio Leitner <fbl at sysclose.org>
  Date:   2021-01-27 (Wed, 27 Jan 2021)

  Changed paths:
    M utilities/ovs-ctl.in

  Log Message:
  -----------
  ovs-ctl: Prepend OVS binary directories to PATH.

The inclusion of LSB functions in ovs-lib resets $PATH to
system's default. Then ovs-ctl appends missing directories
including the OVS default ones $sbindir and $bindir.

The problem is that the wrong binaries can be used if they
are available in the system's default locations because of
the PATH wrong order. The same issue happens if one changes
$OVS_BINDIR or $OVS_SBINDIR variables.

The solution is to prepend those directories if they are not
already in PATH.

Reported-by: Mark Gray <mark.d.gray at redhat.com>
Signed-off-by: Flavio Leitner <fbl at sysclose.org>
Acked-by: Mark Gray <mark.d.gray at redhat.com>
Acked-by: Paolo Valerio <pvalerio at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 9049a11f0110cceb932054bf07df8d719ee8f1c2
      https://github.com/openvswitch/ovs/commit/9049a11f0110cceb932054bf07df8d719ee8f1c2
  Author: Emma Finn <emma.finn at intel.com>
  Date:   2021-01-27 (Wed, 27 Jan 2021)

  Changed paths:
    M tests/system-dpdk.at

  Log Message:
  -----------
  system-dpdk: Fix 4th and 5th testcases of the system-dpdk testsuite.

Testpmd has been renamed to dpdk-testpmd as of DPDK 20.11.
This commit resolves this issue and fixes these tests.

Signed-off-by: Emma Finn <emma.finn at intel.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/1c337c43ac1c...9049a11f0110


More information about the git mailing list