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

[mgmt]: add sonic-mgmt docker for running test in sonic-mgmt repo #606

Merged
merged 2 commits into from
May 18, 2017

Conversation

lguohan
Copy link
Collaborator

@lguohan lguohan commented May 16, 2017

No description provided.

@lguohan lguohan merged commit 6c24574 into sonic-net:master May 18, 2017
@lguohan lguohan deleted the sonic-mgmt branch January 14, 2018 09:07
dgsudharsan pushed a commit to dgsudharsan/sonic-buildimage that referenced this pull request Nov 6, 2019
[warm/fast reboot] continue executing when killing docker failed (sonic-net#713)
[Ying Xie]
[neighbor_advertiser]: Add sleep in setting mirror session and ACL
rules(sonic-net#714)[Shuotian Cheng]
[config]: Flush the neighbor table when removing the router
interface(sonic-net#606) [Shuotian Cheng]
Add a generic configlet application script (sonic-net#716) [Renuka Manavalan]
Management vrf snmp cli support (sonic-net#472) [Harish Venkatraman]
show subinterfaces status (sonic-net#642) [Wenda Ni]
sonic-utilities: Add support for sFlow (sonic-net#592) [Garrick He]
lguohan pushed a commit that referenced this pull request Nov 6, 2019
- [warm/fast reboot] continue executing when killing docker failed (#713) [Ying Xie]
- [neighbor_advertiser]: Add sleep in setting mirror session and ACL rules(#714)[Shuotian Cheng]
- [config]: Flush the neighbor table when removing the router interface(#606) [Shuotian Cheng]
- Add a generic configlet application script (#716) [Renuka Manavalan]
- Management vrf snmp cli support (#472) [Harish Venkatraman]
- show subinterfaces status (#642) [Wenda Ni]
- sonic-utilities: Add support for sFlow (#592) [Garrick He]
zhenggen-xu pushed a commit to zhenggen-xu/sonic-buildimage that referenced this pull request Jan 10, 2020
- [warm/fast reboot] continue executing when killing docker failed (sonic-net#713) [Ying Xie]
- [neighbor_advertiser]: Add sleep in setting mirror session and ACL rules(sonic-net#714)[Shuotian Cheng]
- [config]: Flush the neighbor table when removing the router interface(sonic-net#606) [Shuotian Cheng]
- Add a generic configlet application script (sonic-net#716) [Renuka Manavalan]
- Management vrf snmp cli support (sonic-net#472) [Harish Venkatraman]
- show subinterfaces status (sonic-net#642) [Wenda Ni]
- sonic-utilities: Add support for sFlow (sonic-net#592) [Garrick He]
stepanblyschak pushed a commit to stepanblyschak/sonic-buildimage that referenced this pull request May 10, 2021
…sonic-net#606)

Manually trigger the flush command to remove the deprecated
neighbors that are associated with the removed router interface.

Signed-off-by: Shu0T1an ChenG <shuche@microsoft.com>
saiarcot895 added a commit to saiarcot895/sonic-utilities that referenced this pull request Apr 22, 2024
In the test_po_update test case, one of the things done there is to
remove an IP address from a port channel interface. As part of that, the
current handling for that issues a `ip neigh flush dev ...` command,
added in sonic-net/sonic-buildimage#606, presumably to remove old
neighbor entries that would no longer be valid. I would think that
the kernel would automatically do this, but maybe it didn't back then;
I'm not sure if there's been a behavior change here since then.

In some cases, this command is returning an error, saying "Failed to
send flush request: No such file or directory". I'm not sure why this
is; maybe when iproute2 is going through the list of neighbors, some
neighbor entry was there, but then by the time it issued the deletion
request, that neighbor entry was removed by the kernel since the IP
address was removed. Either way, I don't believe a failure here is
critical.

Therefore, ignore any failures from running this command.

Signed-off-by: Saikrishna Arcot <sarcot@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.

3 participants