Skip to content

Continuous warm reboot test parameters changes#2208

Merged
vaibhavhd merged 1 commit intosonic-net:masterfrom
vaibhavhd:cwb-fail-fix
Sep 11, 2020
Merged

Continuous warm reboot test parameters changes#2208
vaibhavhd merged 1 commit intosonic-net:masterfrom
vaibhavhd:cwb-fail-fix

Conversation

@vaibhavhd
Copy link
Contributor

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Approach

What is the motivation for this PR?

False positive failures due to existing grafeul time for warm-reboot set to 180s whereas it ideal value is 240s.
Failures in iterations do not stop the test (mark fail and continue) - this leads to missed opportunity to debug the issue on the failed-state in DUT.

How did you do it?

Added changes to stop the test when a failure is seen.
Increased warmreboot graceful time from 3 to 4 minutes.

How did you verify/test it?

Tested more than 100 iterations on a T0 device.

Any platform specific information?

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

Documentation

@vaibhavhd vaibhavhd requested a review from a team September 11, 2020 18:30
@vaibhavhd vaibhavhd self-assigned this Sep 11, 2020
@vaibhavhd vaibhavhd merged commit dd0a742 into sonic-net:master Sep 11, 2020
@vaibhavhd vaibhavhd deleted the cwb-fail-fix branch September 25, 2020 16:20
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
* [202012] - sonic-swss submodule update to include following commits:

fca407a (HEAD) [VNET]Fixing nexthop group delete during route change (sonic-net#2198)
a9b6b47 [vxlan] Remove tunnel map objects on VNET tunnel removal (sonic-net#2208)
74e9b9f [FdbOrch] SAI_FDB_EVENT_MOVE generates update with empty update.entry.port_name (sonic-net#2201)
0a99445 [202012][BFD]Registering BFD state change callback during session creation (sonic-net#2203)
aebe4a1 [VS test] skip dpb flaky test (sonic-net#2195) (sonic-net#2207)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants