Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[snmp] LLDPRemManAddrTable does not gets populated if Remote Interface has both ipv4/6 address #7036

Closed
abdosi opened this issue Mar 12, 2021 · 2 comments
Assignees

Comments

@abdosi
Copy link
Contributor

abdosi commented Mar 12, 2021

Issue:
When the Remote LLDP Neighbor advertise both ipv4 and ipv6 address LLDPRemManAddrTable does not gets populated

Eg: below is APP DB where both v4 and v6 address present in lldp_rem_man_addr

  "LLDP_ENTRY_TABLE:Ethernet0": {
    "lldp_rem_port_id_subtype": "5",
    "lldp_rem_sys_cap_supported": "28 00",
    "lldp_rem_index": "1",
    "lldp_rem_chassis_id": "00:11:22:33:44:55",
    "lldp_rem_sys_desc": "I'm a little teapot.",
    "lldp_rem_time_mark": "18545",
    "lldp_rem_sys_cap_enabled": "28 00",
    "lldp_rem_port_desc": " ",
    "lldp_rem_chassis_id_subtype": "4",
    "lldp_rem_sys_name": "switch13",
    "lldp_rem_port_id": "Ethernet1",
    "lldp_rem_man_addr": "10.224.25.100,2603:10e2:290:5016::"
  },

It will fail with below error:
syslog:Mar 11 19:49:18.570078 str-sxxxx-on-2 WARNING snmp#snmp-subagent [sonic_ax_impl] WARNING: Invalid mgmt IP 10.224.25.100,2603:10e2:290:5016:

Expectation:

@abdosi
Copy link
Contributor Author

abdosi commented Mar 12, 2021

working on fix.

@abdosi abdosi self-assigned this Mar 12, 2021
abdosi added a commit to abdosi/sonic-snmpagent that referenced this issue Mar 12, 2021
sonic-net/sonic-buildimage#7036
sonic-net/sonic-buildimage#6636

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
abdosi added a commit to sonic-net/sonic-snmpagent that referenced this issue Mar 15, 2021
…ciated with interface. (#201)

Fixes:-
sonic-net/sonic-buildimage#7036
sonic-net/sonic-buildimage#6636

Updated to use prefix of 16 byte instead of 6 byte for Ipv6 Management Address. Confirmed with other Vendor Nos

Fixed lookup() so that get-next work correctly from test-cases.

Updated all files to use new utility API ip2byte_tuple()
abdosi added a commit to sonic-net/sonic-snmpagent that referenced this issue Mar 15, 2021
…ciated with interface. (#201)

Fixes:-
sonic-net/sonic-buildimage#7036
sonic-net/sonic-buildimage#6636

Updated to use prefix of 16 byte instead of 6 byte for Ipv6 Management Address. Confirmed with other Vendor Nos

Fixed lookup() so that get-next work correctly from test-cases.

Updated all files to use new utility API ip2byte_tuple()
@abdosi
Copy link
Contributor Author

abdosi commented Apr 3, 2021

Fixed.

@abdosi abdosi closed this as completed Apr 3, 2021
praveen-li pushed a commit to praveen-li/sonic-snmpagent that referenced this issue Jun 2, 2021
…ciated with interface. (sonic-net#201)

Fixes:-
sonic-net/sonic-buildimage#7036
sonic-net/sonic-buildimage#6636

Updated to use prefix of 16 byte instead of 6 byte for Ipv6 Management Address. Confirmed with other Vendor Nos

Fixed lookup() so that get-next work correctly from test-cases.

Updated all files to use new utility API ip2byte_tuple()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant