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

Build snmpd with ipv6 support and enable it #3500

Merged
merged 1 commit into from
Sep 24, 2019

Conversation

qiluo-msft
Copy link
Collaborator

@qiluo-msft qiluo-msft commented Sep 21, 2019

- What I did
Fixed #2196

- How I did it

- How to verify it
Tested in DUT

admin@sonic:~/snmp$ docker exec -it snmp bash
root@sonic:/# snmpwalk -v2c -c public ::1 iso.3.6.1.2.1.4.24.4.1.1
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.1 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.5 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.9 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.13 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.17 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.21 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.25 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.0.0.0.0.0.0.0.0.0.10.0.0.29 = IpAddress: 0.0.0.0
iso.3.6.1.2.1.4.24.4.1.1.10.1.0.32.255.255.255.255.0.0.0.0.0 = IpAddress: 10.1.0.32

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@kannankvs
Copy link
Collaborator

@qiluo-msft @lguohan : This was due to the prototype mismatch issue that originated from the net-snmp patch code https://sourceforge.net/p/net-snmp/patches/1376/ which we took as the base for our 0007 patch. The issue is corrected now and I raised the PR3504. Request you to merge the PR3504 first and then merge this PR.

Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
@qiluo-msft qiluo-msft marked this pull request as ready for review September 24, 2019 21:55
@qiluo-msft qiluo-msft merged commit 75907f1 into sonic-net:master Sep 24, 2019
@qiluo-msft qiluo-msft deleted the qiluo/snmp6 branch September 24, 2019 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[snmp] enable-ipv6 is not added when configure net-snmp
5 participants