Skip to content

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

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
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 <[email protected]>
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 <[email protected]>

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

Signed-off-by: Zhenggen Xu <[email protected]>

* 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 <[email protected]>
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 <[email protected]>
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)
mssonicbld added a commit that referenced this issue Mar 22, 2025
@mssonicbld
[submodule] Update submodule sonic-swss-common to the latest HEAD aut…
d8fe086 
…omatically (#21989)

#### Why I did it
src/sonic-swss-common
```
* c82811f - (HEAD -> master, origin/master, origin/HEAD) Added Icmp Echo session table names (#980) (3 hours ago) [manamand2020]
* 38da2cc - Merge pull request #990 from xwjiang-ms/update_start_sh (4 days ago) [xwjiang-ms]
|\ 
| failure_prs.log 03ae013 - update start.sh for new sonic-swss soft bfd tests (4 days ago) [xiaweijiang]
|/ 
* dce55c3 - add DASH_HA_SET_TABLE and DASH_HA_SCOPE_TABLE #985 (11 days ago) [Jing Zhang]
```
#### How I did it
#### How to verify it
#### Description for the changelog
yanjundeng pushed a commit to yanjundeng/sonic-buildimage that referenced this issue Apr 23, 2025
@mssonicbld @yanjundeng
[submodule] Update submodule sonic-swss-common to the latest HEAD aut…
d9dd30e 
…omatically (sonic-net#21989)

#### Why I did it
src/sonic-swss-common
```
* c82811f - (HEAD -> master, origin/master, origin/HEAD) Added Icmp Echo session table names (sonic-net#980) (3 hours ago) [manamand2020]
* 38da2cc - Merge pull request sonic-net#990 from xwjiang-ms/update_start_sh (4 days ago) [xwjiang-ms]
|\ 
| failure_prs.log 03ae013 - update start.sh for new sonic-swss soft bfd tests (4 days ago) [xiaweijiang]
|/ 
* dce55c3 - add DASH_HA_SET_TABLE and DASH_HA_SCOPE_TABLE sonic-net#985 (11 days ago) [Jing Zhang]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit to DavidZagury/sonic-buildimage that referenced this issue Apr 28, 2025
@mssonicbld
[action] [PR:22067] [Mellanox] Update SN56xx relevant SKUs buffers_de…
a2c3acf 
…faults_objects.j2 (sonic-net#990)

<!--
 Please make sure you've read and understood our contributing guidelines:
 https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md

 failure_prs.log skip_prs.log Make sure all your commits include a signature generated with `git commit -s` **

 If this is a bug fix, make sure your description includes "fixes #xxxx", or
 "closes #xxxx" or "resolves #xxxx"

 Please provide the following information:
-->

#### Why I did it
To have buffers updated to new requirements to following SKUs:
Mellanox-SN5600-C256S1
Mellanox-SN5600-C224O8
Mellanox-SN5610N-C256S2
Mellanox-SN5610N-C224O8
Mellanox-SN5640-C512S2
Mellanox-SN5600-C448O16

sonic-mgmt test update: sonic-net/sonic-mgmt#17788

##### Work item tracking
- Microsoft ADO **(number only)**:

#### How I did it
Set port.alpha to 1/64.
Set queue.alpha for queues 1, 2, and 3 to 1.
Set queue.alpha for queue 4 to infinity or 128
#### How to verify it
Load an image with the change on Mellanox system, check SDK dumps and run sonic-mgmt test.
<!--
If PR needs to be backported, then the PR must be tested against the base branch and the earliest backport release branch and provide tested image version on these two branches. For example, if the PR is requested for master, 202211 and 202012, then the requester needs to provide test results on master and 202012.
-->

#### Which release branch to backport (provide reason below if selected)

<!--
- Note we only backport fixes to a release branch, *not* features!
- Please also provide a reason for the backporting below.
- e.g.
- [x] 202006
-->

- [ ] 201811
- [ ] 201911
- [ ] 202006
- [ ] 202012
- [ ] 202106
- [ ] 202111
- [ ] 202205
- [ ] 202211
- [ ] 202305

#### Tested branch (Please provide the tested image version)

<!--
- Please provide tested image version
- e.g.
- [x] 20201231.100
-->

- [ ] <!-- image version 1 -->
- [ ] <!-- image version 2 -->

#### Description for the changelog
<!--
Write a short (one line) summary that describes the changes in this
pull request for inclusion in the changelog:
-->

<!--
 Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.
-->

#### Link to config_db schema for YANG module changes
<!--
Provide a link to config_db schema for the table for which YANG model
is defined
Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md
-->

#### A picture of a cute animal (not mandatory but encouraged)
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