Stop teamd service before syncd (#6755)

- What I did
All SWSS dependent services should stop before SWSS service to avoid future possible issues.
For example 'teamd' service will stop before to allow the driver unload netdev gracefully.
This is to stop all LAG's before restarting syncd service when running 'config reload' command.

- How I did it
Change the order of dependent services of SWSS.

- How to verify it
Run 'config reload' command.
Previously the operation failed when a large number of PortChannel configured on the system.

Signed-off-by: Shlomi Bitton <shlomibi@nvidia.com>
This commit is contained in:
shlomibitton 2021-02-15 16:05:34 +02:00 committed by Danny Allen
parent e0efbc1e14
commit 6361d36fb2

View File

@ -110,14 +110,6 @@ start_peer_and_dependent_services() {
stop_peer_and_dependent_services() { stop_peer_and_dependent_services() {
# if warm/fast start enabled or peer lock exists, don't stop peer service docker # if warm/fast start enabled or peer lock exists, don't stop peer service docker
if [[ x"$WARM_BOOT" != x"true" ]] && [[ x"$FAST_BOOT" != x"true" ]]; then if [[ x"$WARM_BOOT" != x"true" ]] && [[ x"$FAST_BOOT" != x"true" ]]; then
if [[ ! -z $DEV ]]; then
/bin/systemctl stop ${PEER}@$DEV
else
/bin/systemctl stop ${PEER}
fi
for dep in ${DEPENDENT}; do
/bin/systemctl stop ${dep}
done
for dep in ${MULTI_INST_DEPENDENT}; do for dep in ${MULTI_INST_DEPENDENT}; do
if [[ ! -z $DEV ]]; then if [[ ! -z $DEV ]]; then
/bin/systemctl stop ${dep}@$DEV /bin/systemctl stop ${dep}@$DEV
@ -125,7 +117,14 @@ stop_peer_and_dependent_services() {
/bin/systemctl stop ${dep} /bin/systemctl stop ${dep}
fi fi
done done
for dep in ${DEPENDENT}; do
/bin/systemctl stop ${dep}
done
if [[ ! -z $DEV ]]; then
/bin/systemctl stop ${PEER}@$DEV
else
/bin/systemctl stop ${PEER}
fi
fi fi
} }