[config] Restart telemetry service upon config (re)load (#992)

Add telemetry service to the list of services to stop/reset-failed/restart during config load/reload operations.
sonic-net · Jul 21, 2020 · 995cf39 · 995cf39
1 parent 82dfe50
commit 995cf39
Showing 1 changed file with 10 additions and 4 deletions.
diff --git a/config/main.py b/config/main.py
@@ -548,8 +548,10 @@ def _get_disabled_services_list():
 
 
 def _stop_services():
+    # This list is order-dependent. Please add services in the order they should be stopped
     # on Mellanox platform pmon is stopped by syncd
     services_to_stop = [
+        'telemetry',
         'restapi',
         'swss',
         'lldp',
@@ -566,30 +568,33 @@ def _stop_services():
 
 
 def _reset_failed_services():
+    # This list is order-independent. Please keep list in alphabetical order
     services_to_reset = [
         'bgp',
         'dhcp_relay',
         'hostcfgd',
         'hostname-config',
         'interfaces-config',
         'lldp',
+        'nat',
         'ntp-config',
         'pmon',
         'radv',
+        'restapi',
         'rsyslog-config',
+        'sflow',
         'snmp',
         'swss',
         'syncd',
         'teamd',
-        'nat',
-        'sflow',
-        'restapi'
+        'telemetry'
     ]
 
     execute_systemctl(services_to_reset, SYSTEMCTL_ACTION_RESET_FAILED)
 
 
 def _restart_services():
+    # This list is order-dependent. Please add services in the order they should be started
     # on Mellanox platform pmon is started by syncd
     services_to_restart = [
         'hostname-config',
@@ -603,7 +608,8 @@ def _restart_services():
         'hostcfgd',
         'nat',
         'sflow',
-        'restapi'
+        'restapi',
+        'telemetry'
     ]
 
     disable_services = _get_disabled_services_list()