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

Get port alias but not port name in snmp_interface test #990

Closed
okanchou9 opened this issue Sep 27, 2017 · 6 comments
Closed

Get port alias but not port name in snmp_interface test #990

okanchou9 opened this issue Sep 27, 2017 · 6 comments

Comments

@okanchou9
Copy link

Description
Expect get port name but get port alias in snmp_interface test

Steps to reproduce the issue:

  1. Run snmp_interface test

Describe the results you received:
"snmp_intf": [
"Ethernet20/1",
"Ethernet16/1",
"Ethernet30/1",
"Ethernet23/1",
"Ethernet19/1",
"Ethernet17/1",
"Ethernet29/1",
"Ethernet18/1",
"Ethernet24/1",
"Ethernet21/1",
"Ethernet14/1",
"Ethernet32/1",
"Ethernet22/1",
"Ethernet7/1",
"Ethernet4/1",
"Ethernet26/1",
"Ethernet6/1",
"Ethernet13/1",
"Ethernet9/1",
"Ethernet15/1",
"Ethernet12/1",
"Ethernet10/1",
"Ethernet8/1",
"Ethernet31/1",
"Ethernet11/1",
"Ethernet1/1",
"Ethernet28/1",
"Ethernet2/1",
"Ethernet25/1",
"Ethernet3/1",
"Ethernet27/1",
"Ethernet5/1",
]

Describe the results you expected:
"minigraph_map_sonic_to_ngs": {
"Ethernet0": "Ethernet0",
"Ethernet100": "Ethernet100",
"Ethernet104": "Ethernet104",
"Ethernet108": "Ethernet108",
"Ethernet112": "Ethernet112",
"Ethernet116": "Ethernet116",
"Ethernet12": "Ethernet12",
"Ethernet120": "Ethernet120",
"Ethernet124": "Ethernet124",
"Ethernet16": "Ethernet16",
"Ethernet20": "Ethernet20",
"Ethernet24": "Ethernet24",
"Ethernet28": "Ethernet28",
"Ethernet32": "Ethernet32",
"Ethernet36": "Ethernet36",
"Ethernet4": "Ethernet4",
"Ethernet40": "Ethernet40",
"Ethernet44": "Ethernet44",
"Ethernet48": "Ethernet48",
"Ethernet52": "Ethernet52",
"Ethernet56": "Ethernet56",
"Ethernet60": "Ethernet60",
"Ethernet64": "Ethernet64",
"Ethernet68": "Ethernet68",
"Ethernet72": "Ethernet72",
"Ethernet76": "Ethernet76",
"Ethernet8": "Ethernet8",
"Ethernet80": "Ethernet80",
"Ethernet84": "Ethernet84",
"Ethernet88": "Ethernet88",
"Ethernet92": "Ethernet92",
"Ethernet96": "Ethernet96"
}

Additional information you deem important (e.g. issue happens only occasionally):
I'm using version 201709 in my box to run the test, not sure this is expected result at this version or not since I can pass this test at version 201705.
Any comment are welcome, thanks.

Regards,
Kenie Liu
@stcheng
Copy link
Contributor

stcheng commented Sep 27, 2017

@qiluo-msft do we expect the alias or the interface name in the snmp test?

@qiluo-msft
Copy link
Collaborator

I think alias.

@stcheng
Copy link
Contributor

stcheng commented Sep 27, 2017

then the snmp behavior is correct, it is the test issue that fails this task.

@okanchou9
Copy link
Author

Hi all,

Thank you for the info for this behavior.
Also just found that lldp had the same behavior:

ARISTA01T1#sh lldp neighbors
Last table change time : 4:07:16 ago
Number of table inserts : 1
Number of table deletes : 0
Number of table drops : 0
Number of table age-outs : 0

Port Neighbor Device ID Neighbor Port ID TTL
Et1 switch2 Ethernet29/1 120
ARISTA01T1#

Is that also expected, too?

Regards,
Kenie Liu

@taoyl-ms
Copy link
Contributor

Yes. alias is also expected in lldp test.

@okanchou9
Copy link
Author

Thank you guys.

zhenggen-xu added a commit to zhenggen-xu/sonic-buildimage that referenced this issue Aug 2, 2019
@zhenggen-xu
Update sonic-swss submodule
aa0b243 
* src/sonic-swss 63afbd5...5be3963 (11):
  > Remove the platform check for remove/add port action at init phase. (sonic-net#1006)
  > [test]: Add Class object in each test script file (sonic-net#992)
  > [vstest]: add --imgname option to specify vs image to test (sonic-net#996)
  > [qosorch]: Add cir and pir parameters for scheduler (sonic-net#991)
  > [vstest]: report error when team kernel module is not available (sonic-net#994)
  > Support per buffer pool watermark polling mode (sonic-net#973)
  > [test]: Update test_vlan to use dvs functions (sonic-net#986)
  > Add synchronous switch to orch agent (sonic-net#987)
  > ARM32 bit fixes, for 64bit printf format specifier (sonic-net#989)
  > [portsorch]: Rename function isPortReady() to allPortsReady() (sonic-net#990)
  > [routeorch]: Fix observer detaching procedure for next hop observers (sonic-net#985)

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
lguohan pushed a commit that referenced this issue Aug 7, 2019
@zhenggen-xu @lguohan
[Feature: DynamicPortBreakout] Use consolidated bcm file for Seastone…
6f40933 
… platform (#3240)

* Utilize the same sai.profile and config.bcm file

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>

* Fix the index issues for HWSKU: Seastone-DX010-25-50 and Seastone-DX010-50-40

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>

* Update sonic-swss submodule

* src/sonic-swss 63afbd5...5be3963 (11):
  > Remove the platform check for remove/add port action at init phase. (#1006)
  > [test]: Add Class object in each test script file (#992)
  > [vstest]: add --imgname option to specify vs image to test (#996)
  > [qosorch]: Add cir and pir parameters for scheduler (#991)
  > [vstest]: report error when team kernel module is not available (#994)
  > Support per buffer pool watermark polling mode (#973)
  > [test]: Update test_vlan to use dvs functions (#986)
  > Add synchronous switch to orch agent (#987)
  > ARM32 bit fixes, for 64bit printf format specifier (#989)
  > [portsorch]: Rename function isPortReady() to allPortsReady() (#990)
  > [routeorch]: Fix observer detaching procedure for next hop observers (#985)

Signed-off-by: Zhenggen Xu <zxu@linkedin.com>
madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this issue Mar 23, 2020
@wendani @stcheng
[portsorch]: Rename function isPortReady() to allPortsReady() (sonic-…
5d04f05 
…net#990)

Signed-off-by: Wenda Ni <wenni@microsoft.com>
prsunny pushed a commit that referenced this issue Jan 21, 2022
@shi-su
Update sairedis submodule (#9819)
c30f667 
Commits:

02dea43 [202012] Add support for BFD notification (#990)
337fd97 Change sonic-buildimage.vs artifact source from CI build to official build. (#992)
490f442 Support for cisco-8000 platform for sonic-sairedis/syncd (#823)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@stcheng @qiluo-msft @okanchou9 @taoyl-ms