From eb99648a4e4b02285e9e2e14dccd80475cf19be8 Mon Sep 17 00:00:00 2001 From: Gokulnath-Raja <87805083+Gokulnath-Raja@users.noreply.github.com> Date: Sat, 6 May 2023 04:41:12 +0530 Subject: [PATCH] [sflow] Exception handling for if_nametoindex (#11437) (#14457) catch system error and log as warning level instead of error level in case interface was already deleted Signed-off-by: Gokulnath-Raja --- dockers/docker-sflow/port_index_mapper.py | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/dockers/docker-sflow/port_index_mapper.py b/dockers/docker-sflow/port_index_mapper.py index ee9dba15a4..e232d0e383 100755 --- a/dockers/docker-sflow/port_index_mapper.py +++ b/dockers/docker-sflow/port_index_mapper.py @@ -45,7 +45,15 @@ class PortIndexMapper(object): index = port_util.get_index_from_str(ifname) if op == 'SET' and index is None: return - ifindex = if_nametoindex(ifname) + + # catch system error and log as warning level instead of + # error level in case interface was already deleted + ifindex = None + try: + ifindex = if_nametoindex(ifname) + except OSError as e: + logger.log_warning("%s" % str(e)) + if op == 'SET' and ifindex is None: return