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

[swss] Start Arp Update Process #5391

Merged

Conversation

tahmed-dev
Copy link
Contributor

@tahmed-dev tahmed-dev commented Sep 17, 2020

Arp update process was not being started due to an issue with
the directory name having an extra 'd' in supervisor as in
'/etc/supervisord/conf.d/arp_update.conf'.

Fixes: #5217
Closes: #5217

signed-off-by: Tamer Ahmed tamer.ahmed@microsoft.com

- Why I did it
arp_update script is not running

- How I did it
Fixed a bug with the dir name
/etc/supervisord/conf.d/ --> /etc/supervisor/conf.d/

- How to verify it
After reboot, arp_update process should be running

admin@str-s6000-acs-14:~$ sudo zgrep arp_update /var/log/syslog*
Sep 17 01:32:29.345680 str-s6000-acs-14 INFO swss#supervisord 2020-09-17 01:32:25,607 INFO Included extra file "/etc/supervisor/conf.d/arp_update.conf" during parsing
Sep 17 01:33:09.656047 str-s6000-acs-14 INFO swss#supervisord 2020-09-17 01:33:02,318 INFO spawned: 'arp_update' with pid 90
Sep 17 01:33:09.656047 str-s6000-acs-14 INFO swss#supervisord 2020-09-17 01:33:03,411 INFO success: arp_update entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
Sep 17 01:33:23.798435 str-s6000-acs-14 INFO swss#supervisord: arp_update ping6: Warning: source address might be selected on device other than Vlan1000.
admin@str-s6000-acs-14:~$ ps -ef | grep arp_update
root      3870  2435  0 01:33 pts/0    00:00:00 /bin/bash /usr/bin/arp_update
admin     5056  3409  0 01:34 pts/0    00:00:00 grep arp_update

- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

Arp update process was not being started due to an issue with
the directory name having an extra 'd' in supervisor as in
'/etc/supervisord/conf.d/arp_update.conf'.

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
@lguohan
Copy link
Collaborator

lguohan commented Sep 17, 2020

you mean fix #5217?

@lguohan lguohan linked an issue Sep 17, 2020 that may be closed by this pull request
@tahmed-dev
Copy link
Contributor Author

you mean fix #5217?

I am awaiting @stepanblyschak confirmation. Will change it to fixes/closes #5217 once confirmed.

@tahmed-dev tahmed-dev merged commit 7515aea into sonic-net:master Sep 18, 2020
abdosi pushed a commit that referenced this pull request Sep 19, 2020
Arp update process was not being started due to an issue with
the directory name having an extra 'd' in supervisor as in
'/etc/supervisord/conf.d/arp_update.conf'.

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
Arp update process was not being started due to an issue with
the directory name having an extra 'd' in supervisor as in
'/etc/supervisord/conf.d/arp_update.conf'.

signed-off-by: Tamer Ahmed <tamer.ahmed@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.

[fast-reboot] ARP entries are not restored after fast-reboot
4 participants