JIRA-JRTC-7979: [neighsync] Modifcatoin for ignoring neighbor entries #15
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JIRA-JRTC-7979: [neighsync] Ignore neighbor entries on non-router-interface Change-Id: I403ea67142ac482181c6298ec6c560d361cd205b JIRA-SONIC-4938: Don't ignore reserved IPv4 LL address - For BGP unnumber, reserved IPv4 LL (169.254.0.1) will be used as next hop. FRR configure it as permanent neighbor when BGP session is established. We ignore neighbor entry learned on non-router interface currently, this reserved IPv4 LL will not be synced if we don't configure use-link-local-only before this neighbor learnt event. Change-Id: I21ba7b15ea2437747be95e4ccf234db05fb9601c JIRA-SONIC-6815: [neighsyncd] Ignore null MAC address neighbor while adding neighbor By executing the following commands would cause the netlink message bring a NULL MAC neighbor and cause the neighsyncd exception. The libnl api `nl_addr2str` would output the string `"none"` if the lladdr is `NULL`, and the string is an exception case for the class `MacAddress`. Therefore, this commit add to ignore this condition while the `rtnl_neigh_get_lladdr` can not retrieve the lladdr successfully ``` ip neigh replace 1.2.3.8 lladdr 00:33:55:66:55:66 dev Vlan10 nud noarp ip neigh replace 1.2.3.8 lladdr 00:33:55:66:55:66 dev Vlan10 nud none ```
JIRA-SONIC-1982: [Subport] Keep vlan tag when add member port into portchannel of subport When add member port into portchannel, if the portchannel has subport interface, the port should keep the vlan tag. JIRA-SONIC-2085: Handle HostIntfsStripTag properly 1. removeBridgePort should check port.m_child_ports.empty() before HostIntfsStripTag(,SAI_HOSTIF_VLAN_TAG_STRIP). JIRA-JRTC-6756: fix ERR syslog while test_subport setup/teardown fix 2: config portchannel add PortChannel1 config interface ip add PortChannel1.40 100.40.40.40/24 config interface ip remove PortChannel1.40 100.40.40.40/24 config portchannel del PortChannel1 syslog: Dec 27 15:01:38.336049 as5812-54x NOTICE swss#orchagent: :- removeRouterIntfs: Router interface is still referenced Dec 27 15:01:38.337833 as5812-54x ERR swss#orchagent: :- meta_generic_validation_remove: object 0x2000000000a81 reference count is 1, can't remove Dec 27 15:01:38.338038 as5812-54x ERR swss#orchagent: :- removeLag: Failed to remove LAG PortChannel1 lid:2000000000a81 root cause: portchannel is still reference by subport solution: PortsOrch::doLagTask, op == DEL_COMMAND ===== fix 3: syslog: Dec 24 17:28:01.030970 as5812-54x ERR swss#orchagent: :- meta_sai_validate_oid: oid is set to null object id on SAI_OBJECT_TYPE_LAG_MEMBER Dec 24 17:28:01.031218 as5812-54x ERR swss#orchagent: :- setCollectionOnLagMember: Failed to enable collection on LAG member Ethernet28 root cause: PortsOrch::setCollectionOnLagMember/ setDistributionOnLagMember, check lag member id
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JIRA-JRTC-7979: [neighsync] Ignore neighbor entries on non-router-interface
Change-Id: I403ea67142ac482181c6298ec6c560d361cd205b
JIRA-SONIC-4938: Don't ignore reserved IPv4 LL address
Change-Id: I21ba7b15ea2437747be95e4ccf234db05fb9601c
JIRA-SONIC-6815: [neighsyncd] Ignore null MAC address neighbor while adding neighbor
By executing the following commands would cause the netlink message bring a NULL MAC neighbor and cause the neighsyncd exception.
The libnl api
nl_addr2strwould output the string"none"if the lladdr isNULL, and the string is an exception case for the classMacAddress. Therefore, this commit add to ignore this condition while thertnl_neigh_get_lladdrcan not retrieve the lladdr successfullyWhat I did
Why I did it
How I verified it
Details if related