Skip to content

[test]: Add mirror session warm reboot test#1097

Closed
stcheng wants to merge 1 commit intosonic-net:masterfrom
stcheng:test_mirror
Closed

[test]: Add mirror session warm reboot test#1097
stcheng wants to merge 1 commit intosonic-net:masterfrom
stcheng:test_mirror

Conversation

@stcheng
Copy link
Copy Markdown
Contributor

@stcheng stcheng commented Oct 15, 2019

This test validates the post baking stage that recovers
the monitor port picked before the warm reboot.

Signed-off-by: Shu0T1an ChenG shuche@microsoft.com

[test]: Add mirror session warm reboot test
5d1adef 
This test validates the post baking stage that recovers
the monitor port picked before the warm reboot.

Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
@stcheng
Copy link
Copy Markdown
Contributor Author

stcheng commented Oct 15, 2019 

shuotian@debian:sonic/swss/tests  master ✗                                                                                                                                                                                                                                                                         0h23m ⚑ ◒
▶ sudo pytest -s -v --dvsname=vs test_warm_reboot.py::TestWarmReboot::test_swss_mirror_post_baking -s
===================================================================================================================================================== test session starts =====================================================================================================================================================
platform linux2 -- Python 2.7.16, pytest-3.3.0, py-1.5.4, pluggy-0.6.0 -- /usr/bin/python
cachedir: .cache
rootdir: /home/shuotian/sonic/swss/tests, inifile:
collected 1 item

test_warm_reboot.py::TestWarmReboot::test_swss_mirror_post_baking remove extra link dummy
Ethernet20
Ethernet16
Ethernet12
Ethernet20
Ethernet16
Ethernet12
PASSED                                                                                                                                                                                                                                                [100%]

================================================================================================================================================== 1 passed in 78.39 seconds ==================================================================================================================================================

@stcheng
Copy link
Copy Markdown
Contributor Author

stcheng commented Oct 15, 2019

retest this please

daall
daall reviewed Oct 21, 2019
View reviewed changes
@daall
Copy link
Copy Markdown
Contributor

daall commented Oct 21, 2019

Another general question, do these tests cover IPv6 support as well?

daall
daall reviewed Oct 21, 2019
View reviewed changes
@daall daall assigned stcheng and unassigned daall Oct 21, 2019
@daall daall assigned daall and unassigned stcheng Nov 8, 2019
@daall daall mentioned this pull request Nov 13, 2019
Closed
@daall
Copy link
Copy Markdown
Contributor

daall commented Nov 13, 2019

Moving this to #1130

@daall daall closed this Nov 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@daall daall daall left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@daall daall

Labels

Enhancement ➕

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@stcheng @daall