Skip to content

Commit

Permalink
utilities: Fix conntrack flush command
Browse files Browse the repository at this point in the history
This patch fixes two problems.

1) The 'action "ovs-appctl dpctl/flush-conntrack"' does not do
anything because a command does not follow it.

2) Even after adding the command it still doesn't work - it must be
done before the stop_forwarding command.

In addition, make the message more human friendly.

Signed-off-by: Greg Rose <gvrose8192@gmail.com>
Acked-by: Aaron Conole <aconole@redhat.com>
Signed-off-by: Justin Pettit <jpettit@ovn.org>
  • Loading branch information
gvrose8192 authored and justinpettit committed Jun 29, 2018
1 parent e95ce65 commit 265d703
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion utilities/ovs-lib.in
Expand Up @@ -605,6 +605,7 @@ force_reload_kmod () {
stop_ovsdb
start_ovsdb || return 1

action "Flush old conntrack entries" ovs-appctl dpctl/flush-conntrack
stop_forwarding

if action "Saving interface configuration" save_interfaces; then
Expand All @@ -620,7 +621,6 @@ force_reload_kmod () {
for dp in `ovs-dpctl dump-dps`; do
action "Removing datapath: $dp" ovs-dpctl del-dp "$dp"
done
action "ovs-appctl dpctl/flush-conntrack"

ovs_kmod_ctl remove

Expand Down

0 comments on commit 265d703

Please sign in to comment.