From c9a4454450429491c34184d0ceb85eaba62cc525 Mon Sep 17 00:00:00 2001
From: Fernando Royo <froyo@redhat.com>
Date: Thu, 12 Dec 2024 11:01:36 +0100
Subject: [PATCH] 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
---
lib/neutron_plugins/ovn_agent | 26 --------------------------
1 file changed, 26 deletions(-)
diff --git a/lib/neutron_plugins/ovn_agent b/lib/neutron_plugins/ovn_agent
index be3a9e78b2..b7633c8c17 100644
--- a/lib/neutron_plugins/ovn_agent
+++ b/lib/neutron_plugins/ovn_agent
@@ -704,30 +704,6 @@ function _start_ovs {
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 @@ function start_ovn {
# Format logging
setup_logging $OVN_AGENT_CONF
fi
-
- _start_ovn_services
}
function _stop_ovs_dp {