Skip to content

Commit

Permalink
[radv service] radv service should be a cold only dependent of swss (s…
Browse files Browse the repository at this point in the history
…onic-net#3348)

radv should be left alone during warm restart of swss. Otherwise it will
announce departure and cause hosts to lose default gateway.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
  • Loading branch information
yxieca authored and wangshengjun committed Apr 28, 2020
1 parent d3ce652 commit 70c75c4
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions files/build_templates/radv.service.j2
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[Unit]
Description=Router advertiser container
Requires=updategraph.service swss.service
Requires=updategraph.service
After=updategraph.service swss.service
Before=ntp-config.service

Expand All @@ -11,4 +11,4 @@ ExecStart=/usr/bin/{{ docker_container_name }}.sh wait
ExecStop=/usr/bin/{{ docker_container_name }}.sh stop

[Install]
WantedBy=multi-user.target swss.service
WantedBy=multi-user.target
2 changes: 1 addition & 1 deletion files/scripts/swss.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

SERVICE="swss"
PEER="syncd"
DEPENDENT="teamd"
DEPENDENT="teamd radv"
DEBUGLOG="/tmp/swss-syncd-debug.log"
LOCKFILE="/tmp/swss-syncd-lock"

Expand Down

0 comments on commit 70c75c4

Please sign in to comment.