diff options
author | Guru Shetty <guru@ovn.org> | 2016-12-19 04:12:17 -0800 |
---|---|---|
committer | Gurucharan Shetty <guru@ovn.org> | 2016-12-20 08:16:41 -0800 |
commit | d7039b9a68522dc32e51ca02b69fbd1da874d118 (patch) | |
tree | d96153bd3d7d42a70c473089030d9581b7d7ab66 /datapath-windows/ovsext/Conntrack-tcp.c | |
parent | bc759b9e705afe6ced21f8044d874d52ae6c99b6 (diff) | |
download | openvswitch-d7039b9a68522dc32e51ca02b69fbd1da874d118.tar.gz |
ovn-controller: Fix conntrack zone in gateway routers.
The gateway router was using the ct_next action to
reassemble packets. But ct_next action by default would
use the zone allocated for a logical port and in case of
gateway routers that value was zero. This would make
the flow use the default zone of zero. This had some
unintended consequences as the zone used to track packets
and the zone used to eventually commit it (DNAT zone)
was different. As a result, a packet would never have ct.est set.
With this commit, when ct_next action is used in a gateway
router, we use the DNAT zone. This is similar to the
strategy used in commit c2e954a117a8 (ovn-controller: Datapath
based conntrack zone for load-balancing.)
Signed-off-by: Gurucharan Shetty <guru@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'datapath-windows/ovsext/Conntrack-tcp.c')
0 files changed, 0 insertions, 0 deletions