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.

Sign up for GitHub

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

Jump to bottom

[Debian9-S6100]: In LLDP for port id , instead of interface name ,mac address is advertised #1488

Closed
chitra-raghavan opened this issue Mar 13, 2018 · 1 comment
Closed

[Debian9-S6100]: In LLDP for port id , instead of interface name ,mac address is advertised #1488

chitra-raghavan opened this issue Mar 13, 2018 · 1 comment
Assignees
@jleveque
Labels
Bug 🐛 Platform: DellEMC

Comments

@chitra-raghavan
Copy link
Contributor

chitra-raghavan commented Mar 13, 2018 •
edited
Loading

Description

In LLDP for port id , instead of interface name ,mac address is advertised

Steps to reproduce the issue:

  1. Check for LLDP information in peer device

Describe the results you received:
In Peer Arista VM :

ARISTA01T1#show lldp neighbors
Last table change time : 1:55:32 ago
Number of table inserts : 19
Number of table deletes : 18
Number of table drops : 0
Number of table age-outs : 9

Port Neighbor Device ID Neighbor Port ID TTL
Et1 sonic-s6100-01 4c76.25f4.7081 120
ARISTA01T1#

Describe the results you expected:
The sonic DUT should be advertising interface name as the port id.

**Additional information **

Output of show version sonic DUT:

Build : buildimage-brcm-stretch-all 138

root@sonic-s6100-01:~# show ver
SONiC Software Version: SONiC.HEAD.138-b0b8d69
Distribution: Debian 9.4
Kernel: 4.9.0-5-amd64
Build commit: b0b8d69
Build date: Mon Mar 12 13:03:50 UTC 2018
Built by: johnar@jenkins-worker-4

Docker images:
REPOSITORY TAG IMAGE ID SIZE
docker-syncd-brcm HEAD.138-b0b8d69 d92b77a90440 331.6 MB
docker-syncd-brcm latest d92b77a90440 331.6 MB
docker-orchagent-brcm HEAD.138-b0b8d69 ae0ca94d991a 260.5 MB
docker-orchagent-brcm latest ae0ca94d991a 260.5 MB
docker-lldp-sv2 HEAD.138-b0b8d69 0451d4fe2056 270.7 MB
docker-lldp-sv2 latest 0451d4fe2056 270.7 MB
docker-dhcp-relay HEAD.138-b0b8d69 d1c0ab97120f 253.6 MB
docker-dhcp-relay latest d1c0ab97120f 253.6 MB
docker-database HEAD.138-b0b8d69 175362bcb375 252.3 MB
docker-database latest 175362bcb375 252.3 MB
docker-teamd HEAD.138-b0b8d69 d6911b887ad9 257.6 MB
docker-teamd latest d6911b887ad9 257.6 MB
docker-snmp-sv2 HEAD.138-b0b8d69 03312ccf30c8 292 MB
docker-snmp-sv2 latest 03312ccf30c8 292 MB
docker-router-advertiser HEAD.138-b0b8d69 5400a265ea98 250 MB
docker-router-advertiser latest 5400a265ea98 250 MB
docker-platform-monitor HEAD.138-b0b8d69 56e07878c4d7 271.9 MB
docker-platform-monitor latest 56e07878c4d7 271.9 MB
docker-fpm-quagga HEAD.138-b0b8d69 94b4ade9f44d 264.1 MB
docker-fpm-quagga latest 94b4ade9f44d 264.1 MB
@jleveque
Copy link
Contributor

Fixed in PR #1469

dgsudharsan pushed a commit to dgsudharsan/sonic-buildimage that referenced this issue Nov 13, 2020
Updating submodule pointers for sonic-swss and sonic-utilities
e72e27d 
sonic-swss:
[fea7ade] [orchagent][port] In case of successful port creation set log level to NOTICE instead of ERR (sonic-net#1500)
[7b76d2e] Copp Manager Changes (sonic-net#1333)
[bed7970] [orchagent] Arm 32-bit arch compilation warning Fixes (sonic-net#1488)
[b9084a7] Revert: swss: flush g_asicState after each event is done sonic-net#570 (sonic-net#1478)
[d6e15e9] [dvs] Clean-up conftest.py (sonic-net#1406)

sonic-utilities:
[3f15835] [sflow] Add mVRF support (sonic-net#1230)
[dadfc18] Copp Changes (sonic-net#1004)
[b226159] [CLI][PFCWD] Fix issue with specifying ports in pfcwd start on masic platforms (sonic-net#1203)
[40377d3] [chassis]: Configure and show for platform chassis_modules (sonic-net#1145)
[d5eb2f8] [consutil][connect] Remove root need from connect line command (sonic-net#1225)
[6cbec6b] [multi-ASIC] CRM show/config commands changes for multi-asic (sonic-net#1127)
@dgsudharsan dgsudharsan mentioned this issue Nov 13, 2020
Merged
3 tasks
dgsudharsan pushed a commit to dgsudharsan/sonic-buildimage that referenced this issue Nov 16, 2020
Updating submodule pointer for sonic-swss
1dbc830 
sonic-swss:
[fea7ade] [orchagent][port] In case of successful port creation set log
level to NOTICE instead of ERR (sonic-net#1500)
[7b76d2e] Copp Manager Changes (sonic-net#1333)
[bed7970] [orchagent] Arm 32-bit arch compilation warning Fixes (sonic-net#1488)
[b9084a7] Revert: swss: flush g_asicState after each event is done sonic-net#570
(sonic-net#1478)
[d6e15e9] [dvs] Clean-up conftest.py (sonic-net#1406)
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this issue Feb 5, 2022
@Sabareesh-Kumar-Anandan
[orchagent] Arm 32-bit arch compilation warning Fixes (sonic-net#1488)
bed7970 
**What I did**
Replaced %l variants with PRI format specifier to fix compilation warnings for arm 32-bit arch

**How I verified it**
Compiled and verified for arm 32-bit arch (marvell-armhf)

* [orchagent] Arm 32-bit arch compilation warning Fixes
* [orchagent] removed explicit type cast
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jleveque jleveque

Labels
Bug 🐛 Platform: DellEMC
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@lguohan @stcheng @jleveque @chitra-raghavan