Skip to content

Commit

Permalink
[sflow] Exception handling for if_nametoindex (#11437)
Browse files Browse the repository at this point in the history
     catch system error and log as warning level instead of
     error level in case interface was already deleted

Signed-off-by: Gokulnath-Raja <Gokulnath_R@dell.com>
  • Loading branch information
Gokulnath-Raja committed Feb 14, 2023
1 parent 8206925 commit fe46c10
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion dockers/docker-sflow/port_index_mapper.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,15 @@ def update_db(self, ifname, op):
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

Expand Down

0 comments on commit fe46c10

Please sign in to comment.