Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fast/warm reboot] improve fast/warm reboot handling code #973

Merged
merged 1 commit into from
Jun 22, 2019

Conversation

yxieca
Copy link
Collaborator

@yxieca yxieca commented Jun 22, 2019

Summary:
Fixes # (issue)

Type of change

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

Approach

How did you do it?

  • Improve the data test warm up code:
    Let the data plane IO stablize for 30 seconds before testing.
    We observed ptf instability causing the test to fail.
  • Remove config_db.json when fast-reboot into a new image.
    We want the new image to reload minigraph in this case.

Signed-off-by: Ying Xie ying.xie@microsoft.com

How did you verify/test it?

- Improve the data test warm up code:
  Let the data plane IO stablize for 30 seconds before testing.
  We observed ptf instability causing the test to fail.
- Remove config_db.json when fast-reboot into a new image.
  We want the new image to reload minigraph in this case.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca merged commit 491918a into sonic-net:master Jun 22, 2019
@yxieca yxieca deleted the fast branch June 22, 2019 01:15
yxieca added a commit that referenced this pull request Jun 22, 2019
- Improve the data test warm up code:
  Let the data plane IO stablize for 30 seconds before testing.
  We observed ptf instability causing the test to fail.
- Remove config_db.json when fast-reboot into a new image.
  We want the new image to reload minigraph in this case.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
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