From: Simon Horman Date: Thu, 20 Aug 2015 23:45:34 +0000 (-0700) Subject: ovs-ctl: do not attempt to restore flows when called with --delete-bridges X-Git-Tag: v2.5.0~714 X-Git-Url: http://git.cascardo.eti.br/?p=cascardo%2Fovs.git;a=commitdiff_plain;h=24605d4e58ff5d74845d9ae9339f723c0f13608f ovs-ctl: do not attempt to restore flows when called with --delete-bridges When called with --delete-bridges saved flows cannot be restored as the bridges to which they belong no longer exist. This results in the following error messages on restart. ovs-ofctl: br0 is not a bridge or a socket Restoring saved flows ... failed! Although there is no effect of this error other than the message it seems worth avoiding. This patch does so by skipping saving of flows when --delete-bridges is in effect. As flows are no longer saved when --delete-bridges is in effect a side-effect of this change is that restart may be faster when there are many flows. Signed-off-by: Simon Horman Signed-off-by: Gurucharan Shetty Acked-by: Ben Pfaff --- diff --git a/utilities/ovs-ctl.in b/utilities/ovs-ctl.in index 99d789c68..918be2c9c 100755 --- a/utilities/ovs-ctl.in +++ b/utilities/ovs-ctl.in @@ -322,6 +322,12 @@ save_ofports_if_required () { esac } +save_flows_if_required () { + if test X"$DELETE_BRIDGES" != Xyes; then + action "Saving flows" ovs_save save-flows "${script_flows}" + fi +} + save_interfaces () { "$datadir/scripts/ovs-save" save-interfaces ${ifaces} \ > "${script_interfaces}" @@ -372,9 +378,7 @@ force_reload_kmod () { action "Detected internal interfaces: $ifaces" true init_restore_scripts - - action "Saving flows" ovs_save save-flows "${script_flows}" - + save_flows_if_required save_ofports_if_required # Restart the database first, since a large database may take a @@ -450,7 +454,7 @@ restart () { if daemon_is_running ovsdb-server && daemon_is_running ovs-vswitchd; then init_restore_scripts save_interfaces_if_required - action "Saving flows" ovs_save save-flows "${script_flows}" + save_flows_if_required save_ofports_if_required fi