[PMON] Skip chassis_db_init task on Mellanox simx platform (#9017)

Why I did it "chassis_db_init" task of PMON should be skipped on Mellanox simx platform, since the hardware info which this task is trying to access is not available on simx platforms, It will introduce some error log. How I did it Add the capability for "chassis_db_init" in the template for it can be skipped by adding configuration in "pmon_daemon_control.json". add "skip_chassis_db_init" configuration for simx platforms. use symbol link for "pmon_daemon_control.json" since all the simx platforms share the same configuration How to verify it Build an image and install it on simx platform to check whether "chassis_db_init" task is skipped. Signed-off-by: Kebo Liu <[email protected]>
sonic-net · Oct 24, 2021 · 9c4a7c2 · 9c4a7c2
1 parent bb9c36c
commit 9c4a7c2
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 22 deletions.
diff --git a/device/mellanox/x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json
@@ -3,5 +3,6 @@
         "skip_xcvrd": true,
         "skip_psud": true,
         "skip_pcied": true,
-        "skip_thermalctld": true
+        "skip_thermalctld": true,
+        "skip_chassis_db_init": true
 }
diff --git a/device/mellanox/x86_64-mlnx_msn3700_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn3700_simx-r0/pmon_daemon_control.json
diff --git a/device/mellanox/x86_64-mlnx_msn3700_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn3700_simx-r0/pmon_daemon_control.json
@@ -0,0 +1 @@
+../x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json
diff --git a/device/mellanox/x86_64-mlnx_msn4700_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn4700_simx-r0/pmon_daemon_control.json
diff --git a/device/mellanox/x86_64-mlnx_msn4700_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn4700_simx-r0/pmon_daemon_control.json
@@ -0,0 +1 @@
+../x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json
diff --git a/device/mellanox/x86_64-mlnx_msn4800_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn4800_simx-r0/pmon_daemon_control.json
diff --git a/device/mellanox/x86_64-mlnx_msn4800_simx-r0/pmon_daemon_control.json b/device/mellanox/x86_64-mlnx_msn4800_simx-r0/pmon_daemon_control.json
@@ -0,0 +1 @@
+../x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json
diff --git a/dockers/docker-platform-monitor/docker-pmon.supervisord.conf.j2 b/dockers/docker-platform-monitor/docker-pmon.supervisord.conf.j2
@@ -41,6 +41,7 @@ dependent_startup=true
 dependent_startup_wait_for=rsyslogd:running
 {% endif %}
 
+{% if not skip_chassis_db_init %}
 [program:chassis_db_init]
 command=/usr/local/bin/chassis_db_init
 priority=3
@@ -51,6 +52,7 @@ stderr_logfile=syslog
 startsecs=10
 dependent_startup=true
 dependent_startup_wait_for=rsyslogd:running
+{% endif %}
 
 {% if not skip_sensors and HAVE_SENSORS_CONF == 1 %}
 [program:lm-sensors]
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		../x86_64-mlnx_msn2700_simx-r0/pmon_daemon_control.json