[console][loopback] Add loopback test(echo) for console switch by Blueve · Pull Request #3500 · sonic-net/sonic-mgmt

Blueve · 2021-05-19T05:40:26Z

Description of PR

Summary:
Add loopback test(echo) for console switch

Type of change

Bug fix
Testbed and Framework(new/improvement)
Test case(new/improvement)

Approach

What is the motivation for this PR?

Test data transfer are working as expect.
Verify data can go out through the console switch and come back through the console switch.

How did you do it?

Connect to loopback ports (1-16) via reverse SSH and send a random string and expect receive the same

How did you verify/test it?

Run test on physical DUT.
Only configured line 1 and 2

=== Running tests in groups ===
/usr/local/lib/python2.7/dist-packages/ansible/parsing/vault/__init__.py:44: CryptographyDeprecationWarning: Python 2 is no longer supported by the Python core team. Support for it is now deprecated in cryptography, and will be removed in a future release.
  from cryptography.exceptions import InvalidSignature
================================================================================================== test session starts ===================================================================================================
platform linux2 -- Python 2.7.17, pytest-4.6.5, py-1.9.0, pluggy-0.13.1
ansible: 2.8.12
rootdir: /var/src/internal/tests, inifile: pytest.ini
plugins: forked-1.3.0, xdist-1.28.0, html-1.22.1, metadata-1.10.0, repeat-0.9.1, ansible-2.2.2
collected 16 items

console/test_console_loopback.py::test_console_loopback_echo[1] PASSED                                                                                                                                             [  6%]
console/test_console_loopback.py::test_console_loopback_echo[2] PASSED                                                                                                                                             [ 12%]
console/test_console_loopback.py::test_console_loopback_echo[3] SKIPPED                                                                                                                                            [ 18%]
console/test_console_loopback.py::test_console_loopback_echo[4] SKIPPED                                                                                                                                            [ 25%]
console/test_console_loopback.py::test_console_loopback_echo[5] SKIPPED                                                                                                                                            [ 31%]
console/test_console_loopback.py::test_console_loopback_echo[6] SKIPPED                                                                                                                                            [ 37%]
console/test_console_loopback.py::test_console_loopback_echo[7] SKIPPED                                                                                                                                            [ 43%]
console/test_console_loopback.py::test_console_loopback_echo[8] SKIPPED                                                                                                                                            [ 50%]
console/test_console_loopback.py::test_console_loopback_echo[9] SKIPPED                                                                                                                                            [ 56%]
console/test_console_loopback.py::test_console_loopback_echo[10] SKIPPED                                                                                                                                           [ 62%]
console/test_console_loopback.py::test_console_loopback_echo[11] SKIPPED                                                                                                                                           [ 68%]
console/test_console_loopback.py::test_console_loopback_echo[12] SKIPPED                                                                                                                                           [ 75%]
console/test_console_loopback.py::test_console_loopback_echo[13] SKIPPED                                                                                                                                           [ 81%]
console/test_console_loopback.py::test_console_loopback_echo[14] SKIPPED                                                                                                                                           [ 87%]
console/test_console_loopback.py::test_console_loopback_echo[15] SKIPPED                                                                                                                                           [ 93%]
console/test_console_loopback.py::test_console_loopback_echo[16] SKIPPED                                                                                                                                           [100%]

-------------------------------------------------------------------------------- generated xml file: /var/src/internal/tests/logs/tr.xml ---------------------------------------------------------------------------------
================================================================================================ short test summary info =================================================================================================
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 14 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 8 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 7 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 4 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 15 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 3 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 9 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 6 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 11 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 5 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 10 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 16 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 13 has not configured
SKIPPED [1] /var/src/internal/tests/console/test_console_loopback.py:28: Target line 12 has not configured
========================================================================================= 2 passed, 14 skipped in 145.59 seconds =========================================================================================

Any platform specific information?

N/A

Supported testbed topology if it's a new test case?

T0, Console General Testbed

Documentation

Echo https://github.com/Azure/sonic-mgmt/blob/master/docs/testplan/console/console_test_hld.md#44-loopback-test

lgtm-com · 2021-05-19T06:10:21Z

This pull request introduces 1 alert when merging 5f88cf3 into 4900b2e - view on LGTM.com

new alerts:

1 for Syntax error

…-net#3500) * [console][loopback] Add loopback test(echo) for console switch Signed-off-by: Jing Kan jika@microsoft.com

…lly (sonic-net#21664) #### Why I did it src/sonic-swss ``` * 5031aadc - (HEAD -> 202411, origin/202411) Capability query for MACSEC ACL attribute (sonic-net#3511) (31 hours ago) [mssonicbld] * 4b357e59 - Fix VRF update handling for loopback interfaces in IntfsOrch (sonic-net#3512) (31 hours ago) [mssonicbld] * fe98176b - Add a delay between killing teamd processes (sonic-net#3510) (2 days ago) [mssonicbld] * e967711e - Remove RIF from m_rifsToAdd before deleting it (sonic-net#3499) (6 days ago) [mssonicbld] * 337c9a10 - Optimize counter polling interval by making it more accurate (sonic-net#3500) (6 days ago) [mssonicbld] ``` #### How I did it #### How to verify it #### Description for the changelog

…lly (sonic-net#22228) src/sonic-swss * fceb196d - (HEAD -> 202411, origin/202411) Revert "Optimize counter polling interval by making it more accurate (5 minutes ago) [Kumaresh Perumal] * d88b694d - (origin/kperumal/check) Remove unused string variables in flexcounterorch.cpp (17 hours ago) [Kumaresh Perumal] |\ | * 1e4315e1 - (origin/kperumalbfn-patch-1) Remove unused string variables in flexcounterorch.cpp (17 hours ago) [Kumaresh Perumal] |/ * e82f8c2d - Revert "Optimize counter polling interval by making it more accurate (sonic-net#3500)" (17 hours ago) [Kumaresh Peruma

[console][loopback] Add loopback test(echo) for console switch

5f88cf3

Blueve added 3 commits May 19, 2021 09:00

Fix typo

5d22ba4

Test after console session is ready

a5c9f90

Fix issue where the expect is using regex wrongly

578e4bd

Blueve marked this pull request as ready for review May 20, 2021 06:30

Blueve requested a review from a team as a code owner May 20, 2021 06:30

Blueve requested a review from yxieca May 20, 2021 06:31

yxieca approved these changes May 20, 2021

View reviewed changes

Blueve merged commit 5828f3b into sonic-net:master May 21, 2021

Blueve deleted the dev/jika/lb0 branch May 21, 2021 02:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[console][loopback] Add loopback test(echo) for console switch#3500

[console][loopback] Add loopback test(echo) for console switch#3500
Blueve merged 4 commits intosonic-net:masterfrom
Blueve:dev/jika/lb0

Blueve commented May 19, 2021 •

edited

Loading

Uh oh!

lgtm-com bot commented May 19, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Blueve commented May 19, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description of PR

Type of change

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Uh oh!

lgtm-com bot commented May 19, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Blueve commented May 19, 2021 •

edited

Loading