sonic-mgmt icon indicating copy to clipboard operation
sonic-mgmt copied to clipboard

Published 20 hours ago • verified publisher icon sonic-net

fix test_service_warm_restart: disabled warm-restart for services on test finish

Open antonptashnik opened this issue 2 years ago • 2 comments

Description of PR

Summary: disabled warm-restart for services on test finish Fixes # (issue)

  • fixed https://github.com/sonic-net/sonic-mgmt/issues/6445
  • refactored selection of services to warm-restart

Type of change

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

Back port request

  • [ ] 201911
  • [ ] 202012
  • [ ] 202205

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

antonptashnik avatar Sep 30 '22 08:09 antonptashnik

@ZhaohuiS please review

antonptashnik avatar Oct 26 '22 09:10 antonptashnik

@jcaiMR @vaibhavhd Could you please help with this review? It's warm reboot related. Thanks.

ZhaohuiS avatar Nov 03 '22 05:11 ZhaohuiS

@jcaiMR @vaibhavhd @ZhaohuiS please say if any changes needed for the PR. Thanks in advance!

antonptashnik avatar Nov 23 '22 18:11 antonptashnik

@ZhaohuiS please take a look and merge if possible. Thanks in advance!

antonptashnik avatar Nov 30 '22 13:11 antonptashnik