Skip to content

[console] Add console baud rate test#7929

Merged
yaqiangz merged 1 commit intosonic-net:masterfrom
yaqiangz:azure-master_console_baud_rate
Apr 6, 2023
Merged

[console] Add console baud rate test#7929
yaqiangz merged 1 commit intosonic-net:masterfrom
yaqiangz:azure-master_console_baud_rate

Conversation

@yaqiangz
Copy link
Contributor

Description of PR

Summary:
Fixes # 5399

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Add test case to verify the console's baud rate is 9600

How did you do it?

  1. Verify console baud rate in sonic by cmdfile.
  2. Connect to DUT by console to verify the actual baud rate.
  3. While connecting by console, reboot to aboot/onie to verify baud rate.

How did you verify/test it?

Run tests

collected 3 items

dut_console/test_console_baud_rate.py::test_console_baud_rate_config PASSED                                                                                                                  [ 33%]
dut_console/test_console_baud_rate.py::test_baud_rate_sonic_connect PASSED                                                                                                                   [ 66%]
dut_console/test_console_baud_rate.py::test_baud_rate_boot_connect PASSED                                                                                                                    [100%]

Any platform specific information?

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

Documentation

@yaqiangz yaqiangz requested a review from Blueve March 31, 2023 01:56
@yaqiangz
Copy link
Contributor Author

@Blueve Could you pls help to review this PR?

@yaqiangz
Copy link
Contributor Author

yaqiangz commented Apr 3, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yaqiangz
Copy link
Contributor Author

yaqiangz commented Apr 4, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yaqiangz
Copy link
Contributor Author

yaqiangz commented Apr 4, 2023

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yaqiangz yaqiangz merged commit 99a059a into sonic-net:master Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need new test: verify the console's baud rate is 9600

3 participants