BGP warm reboot script to service #3992
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 4 commits
January 8, 2020 10:14
…ice /usr/local/bin" This reverts commit d16d163.
dzhangalibaba
suggested changes
Jan 8, 2020
- access DB correctly
yxieca
reviewed
Jan 15, 2020
yxieca
reviewed
Jan 15, 2020
yxieca
reviewed
Jan 15, 2020
yxieca
reviewed
Jan 15, 2020
|
synced to latest master, retest this please |
|
@heidinet2007 I am sorry for taking so long. This PR fell out of my radar. One thing I just realized (I apologize for that) is that this change only covers warm reboot. It won't cover fast reboot. I understand that we don't currently have an fast-reboot db flag. That might not be something that we can add immediate. Can you run some test:
Thanks |
added 2 commits
May 18, 2020 14:12
yxieca
reviewed
Jun 3, 2020
yxieca
reviewed
Jun 3, 2020
yxieca
reviewed
Jun 3, 2020
added 5 commits
June 5, 2020 16:15
yxieca
previously approved these changes
Jun 8, 2020
yxieca
reviewed
Jun 9, 2020
…m zebra in case it's not configured
|
retest vsimage please |
|
retest vsimage please |
|
retest vsimage please |
|
retest broadcom please |
yxieca
approved these changes
Jul 15, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move BGP warm reboot script to BGP service at /usr/local/bin
Test
###################### warm not enabled ###########################
admin@PSW-7002:/etc/sonic$ show warm_restart config
name enable timer_name timer_duration
bgp false NULL NULL
system false NULL NULL
1 Jan 23 00:03:40.080681 PSW-7002 INFO systemd[1]: Stopping BGP container...
2 Jan 23 00:03:40.090340 PSW-7002 NOTICE admin: Stopping bgp service...
6 Jan 23 00:03:40.922961 PSW-7002 NOTICE admin: Warm boot flag: bgp false.
7 Jan 23 00:03:40.931723 PSW-7002 NOTICE admin: Stopping bgp service...
4 Jan 23 00:04:48.660858 PSW-7002 INFO systemd[1]: Starting BGP container...
5 Jan 23 00:04:48.674793 PSW-7002 NOTICE admin: Starting bgp service...
10 Jan 23 00:04:51.171567 PSW-7002 NOTICE admin: Warm boot flag: bgp false.
###################### warm enabled ##############################
bgp true NULL NULL
system true NULL NULL
14 Jan 23 00:08:21.557400 PSW-7002 INFO systemd[1]: Stopping BGP container...
15 Jan 23 00:08:21.569107 PSW-7002 NOTICE admin: Stopping bgp service...
16 Jan 23 00:08:22.350832 PSW-7002 NOTICE admin: Warm boot flag: bgp true.
17 Jan 23 00:08:22.358876 PSW-7002 NOTICE admin: Stopping bgp service...
18 Jan 23 00:08:22.368532 PSW-7002 NOTICE admin: Kill zebra first
13 Jan 23 00:09:07.687258 PSW-7002 INFO systemd[1]: Starting BGP container...
14 Jan 23 00:09:07.720972 PSW-7002 NOTICE admin: Starting bgp service...
15 Jan 23 00:09:09.337206 PSW-7002 NOTICE admin: Warm boot flag: bgp true.