Skip to content

Change warm-reboot time limit to 1 second#855

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/warm1
Apr 5, 2019
Merged

Change warm-reboot time limit to 1 second#855
qiluo-msft merged 1 commit intosonic-net:masterfrom
qiluo-msft:qiluo/warm1

Conversation

@qiluo-msft
Copy link
Contributor

@qiluo-msft qiluo-msft commented Apr 3, 2019

Tested with Mellanox DUT.
By definition, warm-reboot data plane disruption should be less than 1 second. If some platform implementation achieve better, we could override it specially.

Copy link
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

This parameter can be overriden with test parameter, we should use test parameter instead of changing code.

@lguohan
Copy link
Contributor

lguohan commented Apr 5, 2019

but the default should 1 second, this is changing the default.

@qiluo-msft qiluo-msft merged commit 2b89da3 into sonic-net:master Apr 5, 2019
@qiluo-msft qiluo-msft deleted the qiluo/warm1 branch April 5, 2019 02:27
yxieca pushed a commit that referenced this pull request Apr 5, 2019
auspham pushed a commit to auspham/sonic-mgmt that referenced this pull request Feb 3, 2026
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