Skip to content

Commit bfe9d94

Browse files
yxiecamssonicbld
authored andcommitted
Delay mux/sflow/snmp timer after interface-config service (sonic-net#14506)
Why I did it All these 3 services started after swss service, which used to start after interface-config service. But sonic-net#13084 remove the time constraints for swss. After that, these 3 services has the chance of start earlier when the inteface-config service is restarting the networking service, which could cause db connect request to fail. How I did it Delay mux/sflow/snmp timer after the interface-config service. How to verify it PR test. Config reload can repro the issue in 1-3 retries. With this change. config reload run 30+ iterations without hitting the issue. Signed-off-by: Ying Xie <[email protected]>
1 parent 8ca2d47 commit bfe9d94

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

files/build_templates/mux.service.j2

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[Unit]
22
Description=MUX Cable Container
33
Requires=database.service updategraph.service swss.service
4-
After=swss.service
4+
After=swss.service interfaces-config.service
55
BindsTo=sonic.target
66
After=sonic.target
77
StartLimitIntervalSec=1200

files/build_templates/sflow.service.j2

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[Unit]
22
Description=sFlow container
33
Requisite=swss.service
4-
After=swss.service syncd.service hostcfgd.service
4+
After=swss.service syncd.service hostcfgd.service interfaces-config.service
55
BindsTo=sonic.target
66
After=sonic.target
77
Before=ntp-config.service

files/build_templates/snmp.timer

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[Unit]
22
Description=Delays snmp container until SONiC has started
33
PartOf=snmp.service
4-
After=swss.service
4+
After=swss.service interfaces-config.service
55

66
[Timer]
77
OnUnitActiveSec=0 sec

0 commit comments

Comments
 (0)