Removing start_ovn_services call
The function _start_ovn is responsible for starting the
OVS/OVN services. However, its final action is a call
to _start_ovn_services, which restarts all OVS/OVN services
without any justified reason.
This patch removes that call to avoid unnecessarily
restarting all OVS/OVN services immediately after they
have been started.
Closes-Bug: #2091614
Change-Id: Ia791ecb734531fa933c570d687ac9224ed6b27e4
diff --git a/lib/neutron_plugins/ovn_agent b/lib/neutron_plugins/ovn_agent
index be3a9e7..b7633c8 100644
--- a/lib/neutron_plugins/ovn_agent
+++ b/lib/neutron_plugins/ovn_agent
@@ -704,30 +704,6 @@
fi
}
-function _start_ovn_services {
- _start_process "$OVSDB_SERVER_SERVICE"
- _start_process "$OVS_VSWITCHD_SERVICE"
-
- if is_service_enabled ovn-northd ; then
- _start_process "$OVN_NORTHD_SERVICE"
- fi
- if is_service_enabled ovn-controller ; then
- _start_process "$OVN_CONTROLLER_SERVICE"
- fi
- if is_service_enabled ovn-controller-vtep ; then
- _start_process "$OVN_CONTROLLER_VTEP_SERVICE"
- fi
- if is_service_enabled ovs-vtep ; then
- _start_process "devstack@ovs-vtep.service"
- fi
- if is_service_enabled q-ovn-metadata-agent neutron-ovn-metadata-agent ; then
- _start_process "devstack@q-ovn-metadata-agent.service"
- fi
- if is_service_enabled q-ovn-agent neutron-ovn-agent ; then
- _start_process "devstack@q-ovn-agent.service"
- fi
-}
-
# start_ovn() - Start running processes, including screen
function start_ovn {
echo "Starting OVN"
@@ -797,8 +773,6 @@
# Format logging
setup_logging $OVN_AGENT_CONF
fi
-
- _start_ovn_services
}
function _stop_ovs_dp {