Added port aliases testing.#29
Closed
antonpatenko wants to merge 3 commits intosonic-net:masterfrom
antonpatenko:master
Closed
Added port aliases testing.#29antonpatenko wants to merge 3 commits intosonic-net:masterfrom antonpatenko:master
antonpatenko wants to merge 3 commits intosonic-net:masterfrom
antonpatenko:master
Conversation
…ng aliases from Redis to ones from port_config file.
antonpatenko
changed the title
- Port name shall be used as alias for port if no alias specified for that port in config-file. - Fixed bug when reading an empty alias
Contributor
|
can you also update the port_config.ini with alias in the sonic-swss repo? |
lguohan
reviewed
Nov 9, 2016
ansible/roles/test/tasks/sonic.yml
Outdated
|
|
||
| ### When calling this basic_route, Chip is setup with proper parameters and | ||
| ### SAI API used by syncd daemon are called. This test is mainly for validation purpose used by external vendors. | ||
| - name: Test hardware configuration |
Contributor
There was a problem hiding this comment.
suggest to have a better name: test port alias configuration, and also rename the playbook name to port-alias.yml, tag name to port-alias
Contributor
Author
|
OK, thanks, I will. From: lguohan [mailto:[email protected]] can you also update the port_config.ini in the sonic-swss repo? — |
Collaborator
|
Hi @antonpatenko, this PR is to add ansible playbook based testing. If this test is still required, probably you can try to add a pytest based script. I am closing this PR for now. |
wangxin
pushed a commit
to wangxin/sonic-mgmt
that referenced
this pull request
Feb 21, 2025
[action] [PR:16657] [T0 T1 merge] update multidut pfcwd tests
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Why I did it Advance sonic-dhcp-relay submodule head on 202012 branch How I did it Added the following commits: a4b15d8 jcaiMR Thu Dec 29 14:18:28 2022 +0800 fix relay-reply dhcpv6 packet counter issue (sonic-net#29) 677543f jcaiMR Sat Dec 17 00:24:32 2022 +0800 fix handleSwssNotification crash in dhcp6relay (sonic-net#28) ed86546 jcaiMR Wed Dec 14 14:08:58 2022 +0800 Fix multiple vlan issue (sonic-net#27) 5ec1f5b Vivek Thu Dec 8 09:44:15 2022 -0800 Made the Error log informative (sonic-net#22) 063d41b jcaiMR Wed Nov 30 14:41:53 2022 +0800 disable cfg dynamic change (sonic-net#25) d4a51f6 kellyyeh Tue Jan 31 18:09:08 2023 -0800 Add unittest infrastructure (sonic-net#5) (sonic-net#31) How to verify it Ran full dhcpv6 test suite on lab device
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Update dhcprelay submodule to include the following commits 4bf1868 fix relay-reply dhcpv6 packet counter issue (sonic-net#29)
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
Why I did it Cherry-pick PR sonic-net#22849 to update the Nokia PMON submodule Update Nokia sonic-platform submodule which includes the commits in branch 202405 and 202503. Include commits: 05abbf6 [IXR7220-D4] Add port leds functionality d4f4e4d Update Fan Drawer index in cpld and fix fan target speed 4800083 [mackinac] warnings in general build c5b3a14 [mackinac] adding prom programming spidev; incidental c++ changes without functional impact fb37c5a [mackinac] working drivers, pcon, sets devices 0c56adf [x3b]Change access permissions for syseeprom 30cbed3 Using PcieUtil utility to check PCIE status 2b41e41 [x1b] modprobe.d/cpuctl.conf f234d5a [x3b] wait for pcon driver sysfs ready 054d682 [x3b]Fix PSU status check issue (sonic-net#31) f6cb50e [X1b] Add support for Nokia X1b platform (sonic-net#30) 921f9dd Adds CI to build the debian (sonic-net#29) 7728ef2 [x4][x1b] consolidating shared cpuctl.ko source code from working x3b version 9f73cf4 Fix the reboot-cause fro upgrading from MFG image 7e057f9 [x3b]Update PSU code to adapt the I2C driver modification c64aba4 Fixed Watchdog and Others reboot-cause 39215bf moving platform_init script output into journal/syslog 4755411 [x3b] combined write/read 3a13b63 Adding bridge code for x3b x1b x4 53a0696 [x3b]Update fan module and thermal algorithm 8052488 [x3b] adds spi bus 0aac79f [x3b]Fix and update fan and thermal modules 169e0d8 Fix reboot-cause for Kernel Panic when kudmp is enabled dd003e1 [IXR7220-D4] FW Upgrades Support e143b5f Revert "Reduce traffic loss when the IMM reboots ungracefully" f7eec2c Revert "Add dependency for IMM ungracefyl reboot service" b345006 [Chassis] Conditionalize stuffing QFPGA into reset based on kernel AER setting 4652166 [Chassis] Delete FP ports shutdown at LC ungraceful reset time Work item tracking Microsoft ADO (number only): How I did it Update Nokia sonic-platform submodule which includes the commits in branch 202405 and 202503. Include commits: 05abbf6 [IXR7220-D4] Add port leds functionality d4f4e4d Update Fan Drawer index in cpld and fix fan target speed 4800083 [mackinac] warnings in general build c5b3a14 [mackinac] adding prom programming spidev; incidental c++ changes without functional impact fb37c5a [mackinac] working drivers, pcon, sets devices 0c56adf [x3b]Change access permissions for syseeprom 30cbed3 Using PcieUtil utility to check PCIE status 2b41e41 [x1b] modprobe.d/cpuctl.conf f234d5a [x3b] wait for pcon driver sysfs ready 054d682 [x3b]Fix PSU status check issue (sonic-net#31) f6cb50e [X1b] Add support for Nokia X1b platform (sonic-net#30) 921f9dd Adds CI to build the debian (sonic-net#29) 7728ef2 [x4][x1b] consolidating shared cpuctl.ko source code from working x3b version 9f73cf4 Fix the reboot-cause fro upgrading from MFG image 7e057f9 [x3b]Update PSU code to adapt the I2C driver modification c64aba4 Fixed Watchdog and Others reboot-cause 39215bf moving platform_init script output into journal/syslog 4755411 [x3b] combined write/read 3a13b63 Adding bridge code for x3b x1b x4 53a0696 [x3b]Update fan module and thermal algorithm 8052488 [x3b] adds spi bus 0aac79f [x3b]Fix and update fan and thermal modules 169e0d8 Fix reboot-cause for Kernel Panic when kudmp is enabled dd003e1 [IXR7220-D4] FW Upgrades Support e143b5f Revert "Reduce traffic loss when the IMM reboots ungracefully" f7eec2c Revert "Add dependency for IMM ungracefyl reboot service" b345006 [Chassis] Conditionalize stuffing QFPGA into reset based on kernel AER setting 4652166 [Chassis] Delete FP ports shutdown at LC ungraceful reset time Signed-off-by: mlok <[email protected]>
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
advance dhcp relay for 202211 4bf1868 - (HEAD, origin/master, origin/HEAD, master) fix relay-reply dhcpv6 packet counter issue (add support for a7050 qx32 platform sonic-net#29) (2 weeks ago) [jcaiMR] 9b30690 - fix handleSwssNotification crash in dhcp6relay (Add libnl-nf-3-200 to docker-team sonic-net#28) (4 weeks ago) [jcaiMR] 047afb7 - Fix multiple vlan issue (Failure trying to run: chroot /sonic-buildimage/fsroot mount -t proc proc /proc sonic-net#27) (4 weeks ago) [jcaiMR] ff6bec3 - Made the Error log informative (add python-tenjin as build dependency for p4-switch sonic-net#22) (5 weeks ago) [Vivek] 2fbe729 - disable cfg dynamic change (p4: fix build dependency for python-p4c-bm sonic-net#25) (6 weeks ago) [jcaiMR] 13d0805 - Use github code scanning instead of LGTM (Removed sx-libnl from Mellanox containers dependencies. sonic-net#26) (6 weeks ago) [Liu Shilong] 1e846f6 - Fix packet range check for relay-reply packets (update sonic-swss and p4-switch submodule to fix docker sonic p4 bug sonic-net#21) (7 weeks ago) [kellyyeh] 4d19e13 - Add unittest infrastructure (Cavium customization for docker containers sonic-net#5) (8 weeks ago) [kellyyeh] 7f4fdab - fix packet range check issue (Makefile: add build dependency for python-p4c-bm sonic-net#20) (9 weeks ago) [jcaiMR] 257ecdf - Add client packet UDP header length check (change port_config.ini directory for s6000 sonic-net#19) (2 months ago) [kellyyeh]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Because of extending of the port_config.ini format to have a port alias for each port, one more field will be read from port_config.ini. This test evaluates if Redis DB has got the same ports aliases as ones in port_config.ini.