[PMON][ethtool] Remove the ethtool from the PMON docker per discussion

Signed-off-by: mlok <marty.lok@nokia.com>
sonic-net · Apr 8, 2024 · c00592a · c00592a
1 parent 74ad45d
commit c00592a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/dockers/docker-platform-monitor/Dockerfile.j2 b/dockers/docker-platform-monitor/Dockerfile.j2
@@ -25,10 +25,10 @@ RUN apt-get update &&   \
         python3-jsonschema \
         libpci3         \
         iputils-ping    \
-        pciutils        \
-        ethtool &&	\
+        pciutils
+
 # smartmontools version should match the installed smartmontools in sonic_debian_extension build template
-    apt-get install -y -t bookworm-backports \
+RUN apt-get install -y -t bookworm-backports \
         smartmontools
 
 # On Arista devices, the sonic_platform wheel is not installed in the container.

diff --git a/rules/docker-platform-monitor.mk b/rules/docker-platform-monitor.mk
@@ -64,7 +64,6 @@ $(DOCKER_PLATFORM_MONITOR)_aboot_RUN_OPT += -v /usr/lib/python3/dist-packages/so
 $(DOCKER_PLATFORM_MONITOR)_BASE_IMAGE_FILES += cmd_wrapper:/usr/bin/sensors
 $(DOCKER_PLATFORM_MONITOR)_BASE_IMAGE_FILES += cmd_wrapper:/usr/sbin/iSmart
 $(DOCKER_PLATFORM_MONITOR)_BASE_IMAGE_FILES += cmd_wrapper:/usr/sbin/SmartCmd
-$(DOCKER_PLATFORM_MONITOR)_BASE_IMAGE_FILES += cmd_wrapper:/usr/bin/ethtool
 $(DOCKER_PLATFORM_MONITOR)_FILES += $(SUPERVISOR_PROC_EXIT_LISTENER_SCRIPT)
 
 SONIC_BOOKWORM_DOCKERS += $(DOCKER_PLATFORM_MONITOR)