From d278e865c64daf642ef2eb937d813cb531daf5e7 Mon Sep 17 00:00:00 2001
From: Lotus Fenn <lotus@nexthop.ai>
Date: Wed, 15 Apr 2026 05:54:55 +0000
Subject: [PATCH] use eth0 override in monit

Signed-off-by: Lotus Fenn <lotus@nexthop.ai>
---
 files/image_config/monit/mgmt_oper_status.py | 60 ++++++++++++++++++--
 1 file changed, 56 insertions(+), 4 deletions(-)

diff --git a/files/image_config/monit/mgmt_oper_status.py b/files/image_config/monit/mgmt_oper_status.py
index f5783347d78..0d6aeb5042c 100755
--- a/files/image_config/monit/mgmt_oper_status.py
+++ b/files/image_config/monit/mgmt_oper_status.py
@@ -2,13 +2,66 @@
 
 """
 """
+import os
 import sys
-import subprocess
 import syslog
 
+import sonic_platform.platform
 from swsscommon.swsscommon import SonicV2Connector
 
 
+def _get_kernel_operstate(intf):
+    """!
+    Read the kernel-reported operstate for a network interface.
+
+    @return 'up', 'down', or other kernel operstate string.
+    """
+    path = "/sys/class/net/{}/operstate".format(intf)
+    if not os.path.exists(path):
+        syslog.syslog(syslog.LOG_ERR,
+                      "operstate path does not exist: {}".format(path))
+        return "unknown"
+    with open(path, "r") as fh:
+        return fh.readline().strip().lower()
+
+
+def _has_platform_mgmt_link_check():
+    """!
+    Check whether this platform overrides the kernel-reported management port
+    link status via the platform API (get_management_port_link_status_override).
+    """
+    try:
+        chassis = sonic_platform.platform.Platform().get_chassis()
+    except Exception:
+        return False
+    return hasattr(chassis, "get_management_port_link_status_override")
+
+
+def _platform_mgmt_link_check(intf):
+    """!
+    Query the platform for the real management port link status via the
+    platform API. If the platform provides an override (returns True/False),
+    use it instead of the kernel operstate. If not implemented (returns None)
+    or on error, return None to fall back to the kernel operstate.
+
+    @return 'up' if the platform reports link up;
+            'down' if the platform reports link down;
+            None if the platform does not override or on error.
+    """
+    try:
+        chassis = sonic_platform.platform.Platform().get_chassis()
+        is_up = chassis.get_management_port_link_status_override(intf)
+    except Exception as e:
+        syslog.syslog(syslog.LOG_WARNING,
+                      "get_management_port_link_status_override failed: %s" % str(e))
+        return None
+    if is_up is None:
+        return None
+    syslog.syslog(syslog.LOG_DEBUG,
+                  "Platform mgmt link check: %s" % ("up" if is_up else "down"))
+    return "up" if is_up else "down"
+
+
 def main():
     db = SonicV2Connector(use_unix_socket_path=True)
     db.connect('CONFIG_DB')
@@ -38,9 +91,8 @@ def main():
 
                 # Update oper status if modified
                 prev_oper_status = state_db_mgmt.get('oper_status', 'unknown')
-                port_operstate_path = '/sys/class/net/{}/operstate'.format(port)
-                oper_status = subprocess.run(['cat', port_operstate_path], capture_output=True, text=True)
-                current_oper_status = oper_status.stdout.strip()
+                override = _platform_mgmt_link_check(port) if _has_platform_mgmt_link_check() else None
+                current_oper_status = override if override is not None else _get_kernel_operstate(port)
                 if current_oper_status != prev_oper_status:
                     db.set(db.STATE_DB, state_db_key, 'oper_status', current_oper_status)
                     log_level = syslog.LOG_INFO if current_oper_status == 'up' else syslog.LOG_WARNING