Skip to content

[sflow] Exception handling for if_nametoindex (#11437)#14457

Merged
yxieca merged 1 commit intosonic-net:202211from
Gokulnath-Raja:sflow_bug#11437_port_index_mapper_interface_handling_202211
May 5, 2023
Merged

[sflow] Exception handling for if_nametoindex (#11437)#14457
yxieca merged 1 commit intosonic-net:202211from
Gokulnath-Raja:sflow_bug#11437_port_index_mapper_interface_handling_202211

Conversation

@Gokulnath-Raja
Copy link
Copy Markdown
Contributor

@Gokulnath-Raja Gokulnath-Raja commented Mar 29, 2023 •
edited by lguohan
Loading

Fix #11437

Why I did it
sflow process exited when failed to convert the interface index from interface name

How I did it
Added exception handling code and logged when OSError exception.

How to verify it
Recreated the bug scenario #11437 and ensured that sflow process not exited.

Description for the changelog
catch system error and log as warning level instead of
error level in case interface was already deleted.

Logs
steps :

root@sonic:~# sudo config vlan member del 4094 PortChannel0001
root@sonic:~# sudo config vlan member del 4094 Ethernet2
root@sonic:~# sudo config vlan del 4094
root@sonic:~#

"WARNING sflow#port_index_mapper: no interface with this name" is  seen but no crash is reported

syslogs :


Jan 23 09:17:24.420448 sonic NOTICE swss#orchagent: :- removeVlanMember: Remove member Ethernet2 from VLAN Vlan4094 lid:ffe vmid:27000000000a53
Jan 23 09:17:24.420710 sonic NOTICE swss#orchagent: :- flushFdbEntries: flush key: SAI_OBJECT_TYPE_FDB_FLUSH:oid:0x21000000000000, fields: 3
Jan 23 09:17:24.420847 sonic NOTICE swss#orchagent: :- recordFlushFdbEntries: flush key: SAI_OBJECT_TYPE_FDB_FLUSH:oid:0x21000000000000, fields: 3
Jan 23 09:17:24.426082 sonic NOTICE syncd#syncd: :- processFdbFlush: fdb flush succeeded, updating redis database
Jan 23 09:17:24.426242 sonic NOTICE syncd#syncd: :- processFlushEvent: received a flush port fdb event, portVid = oid:0x3a000000000a52, bvId = oid:0x26000000000a51
Jan 23 09:17:24.426374 sonic NOTICE syncd#syncd: :- processFlushEvent: pattern ASIC_STATE:SAI_OBJECT_TYPE_FDB_ENTRY:*oid:0x26000000000a51*, portStr oid:0x3a000000000a52
Jan 23 09:17:24.427104 sonic NOTICE bgp#fpmsyncd: :- onRouteMsg: RouteTable del msg for route with only one nh on eth0/docker0: fe80::/64 :: eth0
Jan 23 09:17:24.427182 sonic NOTICE bgp#fpmsyncd: :- onRouteMsg: RouteTable del msg for route with only one nh on eth0/docker0: fd00::/80 :: docker0
Jan 23 09:17:24.428502 sonic NOTICE swss#orchagent: :- meta_sai_on_fdb_flush_event_consolidated: processing consolidated fdb flush event of type: SAI_FDB_ENTRY_TYPE_DYNAMIC
Jan 23 09:17:24.429058 sonic NOTICE swss#orchagent: :- meta_sai_on_fdb_flush_event_consolidated: fdb flush took 0.000606 sec
Jan 23 09:17:24.431496 sonic NOTICE swss#orchagent: :- setHostIntfsStripTag: Set SAI_HOSTIF_VLAN_TAG_STRIP to host interface: Ethernet2
Jan 23 09:17:24.431675 sonic NOTICE swss#orchagent: :- flushFdbEntries: flush key: SAI_OBJECT_TYPE_FDB_FLUSH:oid:0x21000000000000, fields: 2
Jan 23 09:17:24.431797 sonic NOTICE swss#orchagent: :- recordFlushFdbEntries: flush key: SAI_OBJECT_TYPE_FDB_FLUSH:oid:0x21000000000000, fields: 2
Jan 23 09:17:24.437009 sonic NOTICE swss#orchagent: :- meta_sai_on_fdb_flush_event_consolidated: processing consolidated fdb flush event of type: SAI_FDB_ENTRY_TYPE_DYNAMIC
Jan 23 09:17:24.437532 sonic NOTICE swss#orchagent: :- meta_sai_on_fdb_flush_event_consolidated: fdb flush took 0.000514 sec
Jan 23 09:17:24.437942 sonic NOTICE syncd#syncd: :- processFdbFlush: fdb flush succeeded, updating redis database
Jan 23 09:17:24.438065 sonic NOTICE syncd#syncd: :- processFlushEvent: received a flush port fdb event, portVid = oid:0x3a000000000a52, bvId = oid:0x0
Jan 23 09:17:24.438173 sonic NOTICE syncd#syncd: :- processFlushEvent: pattern ASIC_STATE:SAI_OBJECT_TYPE_FDB_ENTRY:*, portStr oid:0x3a000000000a52
Jan 23 09:17:24.440348 sonic NOTICE swss#orchagent: :- removeBridgePort: Remove bridge port Ethernet2 from default 1Q bridgeJan 23 09:17:29.782554 sonic NOTICE swss#orchagent: :- removeVlan: VLAN Vlan4094 still has 1 FDB entries
Jan 23 09:17:29.791373 sonic WARNING sflow#port_index_mapper: no interface with this name

@Gokulnath-Raja
[sflow] Exception handling for if_nametoindex (sonic-net#11437)
c584d0b 
     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>
@Gokulnath-Raja Gokulnath-Raja requested a review from lguohan as a code owner March 29, 2023 09:42
@Gokulnath-Raja Gokulnath-Raja mentioned this pull request Mar 29, 2023
Merged
@vivekrnv
Copy link
Copy Markdown
Contributor

vivekrnv commented Apr 18, 2023

@Gokulnath-Raja can you re-trigger the CI?

@Gokulnath-Raja
Copy link
Copy Markdown
Contributor Author

Gokulnath-Raja commented Apr 19, 2023

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Apr 19, 2023

Commenter does not have sufficient privileges for PR 14457 in repo sonic-net/sonic-buildimage

@keboliu
Copy link
Copy Markdown
Collaborator

keboliu commented Apr 19, 2023

/AzurePipelines run Azure.sonic-buildimage

please try '/azpw run Azure.sonic-buildimage'

@Gokulnath-Raja

@Gokulnath-Raja
Copy link
Copy Markdown
Contributor Author

Gokulnath-Raja commented Apr 19, 2023

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Apr 19, 2023

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Apr 19, 2023

Azure Pipelines successfully started running 1 pipeline(s).

@vivekrnv
Copy link
Copy Markdown
Contributor

vivekrnv commented Apr 19, 2023

@venkatmahalingam Can you review this?

venkatmahalingam
venkatmahalingam reviewed Apr 19, 2023
View reviewed changes
dockers/docker-sflow/port_index_mapper.py
ifindex = None
try:
ifindex = if_nametoindex(ifname)
except OSError as e:
Copy link
Copy Markdown
Collaborator

@venkatmahalingam venkatmahalingam Apr 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have the new code block only for SET operation?

@liat-grozovik
Copy link
Copy Markdown
Collaborator

liat-grozovik commented Apr 27, 2023

thanks for the fix. @lguohan @qiluo-msft could you please merge?

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Apr 27, 2023

this is 202211 branch. @StormLiangMS , can you merge?

@keboliu keboliu requested a review from StormLiangMS May 4, 2023 09:40
@dgsudharsan
Copy link
Copy Markdown
Collaborator

dgsudharsan commented May 4, 2023

This is Manual cherry-pick of #13567

@yxieca yxieca merged commit eb99648 into sonic-net:202211 May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vivekrnv vivekrnv vivekrnv approved these changes

@venkatmahalingam venkatmahalingam venkatmahalingam approved these changes

@lguohan lguohan Awaiting requested review from lguohan lguohan is a code owner

@StormLiangMS StormLiangMS Awaiting requested review from StormLiangMS

Assignees

No one assigned

Labels

Included in 202211 Branch Request for 202211 Branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@Gokulnath-Raja @vivekrnv @keboliu @mssonicbld @liat-grozovik @lguohan @dgsudharsan @venkatmahalingam @yxieca