Skip to content

[cold reboot] Added changes for reboot cause check of cold reboot test case#2702

Merged
yxieca merged 1 commit intosonic-net:masterfrom
OleksandrKozodoi:add_changes_for_reboot_cause_check_of_reboot_test
Dec 24, 2020
Merged

[cold reboot] Added changes for reboot cause check of cold reboot test case#2702
yxieca merged 1 commit intosonic-net:masterfrom
OleksandrKozodoi:add_changes_for_reboot_cause_check_of_reboot_test

Conversation

@OleksandrKozodoi
Copy link
Copy Markdown
Contributor

Signed-off-by: Oleksandr Kozodoi [email protected]

Description of PR

Summary:
Added changes for reboot cause check of cold reboot test case.

Type of change

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

Approach

What is the motivation for this PR?

PR #6130 of sonic-buildimage changed the output of CLI command 'show reboot-cause' and test_cold_reboot was failed.

The old version of CLI command output

$ show reboot-cause                                                                                                                                                     
User issued 'reboot' command [User: , Time: Wed 23 Dec 2020 12:57:33 PM UTC] 

The new version of CLI command output

$ show reboot-cause 
Non-Hardware (reboot, time: 2020-12-23 10:56:50 UTC)

How did you do it?

Added two types of 'show reboot-cause' output to check of cold reboot test case.

How did you verify/test it?

py.test --testbed=testbed-t1 --inventory=../ansible/lab --testbed_file=../ansible/testbed.csv --host-pattern=testbed-t1 -- module-path=../ansible/library platform_tests/test_reboot.py::test_cold_reboot

platform_tests/test_reboot.py::test_cold_reboot PASSED

Any platform specific information?

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

Documentation

@OleksandrKozodoi
Copy link
Copy Markdown
Contributor Author

@yxieca @wangxin Please review

@yxieca yxieca merged commit 9e23d5a into sonic-net:master Dec 24, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Update for swss following commits:
3108985 - 2023-03-28 : [202012][muxorch] handling multiple mux nexthops for route (sonic-net#2702) [Nikola Dancejic]
f412625 - 2023-03-24 : [202012]Custom monitoring based priority tunnels (sonic-net#2671) [siqbal1986]
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.

2 participants