Skip to content

Increase max timeout to 400s for checking critical services during safe reload#10125

Merged
StormLiangMS merged 1 commit intosonic-net:masterfrom
ZhaohuiS:fix/reboot_time
Sep 26, 2023
Merged

Increase max timeout to 400s for checking critical services during safe reload#10125
StormLiangMS merged 1 commit intosonic-net:masterfrom
ZhaohuiS:fix/reboot_time

Conversation

@ZhaohuiS
Copy link
Contributor

@ZhaohuiS ZhaohuiS commented Sep 25, 2023

Description of PR

Summary:
Fixes # (issue)

Type of change

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

Back port request

  • 201911
  • 202012
  • 202205
  • 203205

Approach

What is the motivation for this PR?

Some services boot up for more than 5 mins after config reload on 6100 platform.

How did you do it?

Increase max timeout to 400s for checking critical services for safe reload.

How did you verify/test it?

Run 30 times of arp/test_arpall.py::test_arp_garp_no_update which has config reload in teardown phase, all passed.

Any platform specific information?

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

Documentation

@ZhaohuiS ZhaohuiS changed the title Increate max timeout to 400s for checking critical services Increase max timeout to 400s for checking critical services Sep 25, 2023
@ZhaohuiS ZhaohuiS changed the title Increase max timeout to 400s for checking critical services Increase max timeout to 400s for checking critical services during safe reload Sep 25, 2023
Copy link
Collaborator

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit 66c1a80 into sonic-net:master Sep 26, 2023
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 26, 2023
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202012: #10141

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202205: #10142

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202305: #10143

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.

3 participants