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

[restore_neighbors.py] build arp packet with correct hwsrc and psrc #1158

Merged
merged 1 commit into from
Jan 28, 2020

Conversation

stepanblyschak
Copy link
Contributor

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

$ sudo tcpdump -ni PortChannel0003 --direction out
16:33:40.572995 ARP, Request who-has 10.0.0.61 tell 10.112.71.4, length
28

All neighbors than are stil considered as STALE.

Signed-off-by: Stepan Blyschak [email protected]

What I did

Why I did it

How I verified it

Details if related

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

$ sudo tcpdump -ni PortChannel0003 --direction out
16:33:40.572995 ARP, Request who-has 10.0.0.61 tell 10.112.71.4, length
28

All neighbors than are stil considered as STALE.

Signed-off-by: Stepan Blyschak <[email protected]>
@yxieca
Copy link
Contributor

yxieca commented Jan 6, 2020

@zhenggen-xu looks like since day 1 the smac and srcip were not used for IPv4 ARP packet. Do you remember why? Do you have concern for this change?

@zhenggen-xu
Copy link
Collaborator

What is the test topology and where is PortChannel0003?

scapy supposed to use the interfaces sip and smac when the arp was sent out. It would be helpful to understand why it was not. To add them explicitly should be fine as done in this PR, but might be good to check why, just don't want to hide any potential real issue somewhere.

@stepanblyschak
Copy link
Contributor Author

@zhenggen-xu test topology is T0, PortChannel0003 is there if you deploy minigraph for t0 from sonic-mgmt.
I remember some python library cached the ip/mac on import time. It might be the same with scapy.
If script starts before PortChannels are created and configured in kernel it might use mgmt as a default. That's my understanding.

@liat-grozovik
Copy link
Collaborator

retest vs please

@liat-grozovik
Copy link
Collaborator

retest this please

@prsunny
Copy link
Collaborator

prsunny commented Jan 23, 2020

retest vs please

@daall
Copy link
Contributor

daall commented Jan 25, 2020

retest this please

2 similar comments
@daall
Copy link
Contributor

daall commented Jan 25, 2020

retest this please

@prsunny
Copy link
Collaborator

prsunny commented Jan 27, 2020

retest this please

@prsunny prsunny merged commit 97abec5 into sonic-net:master Jan 28, 2020
prsunny pushed a commit that referenced this pull request Jan 28, 2020
…1158)

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

Signed-off-by: Stepan Blyschak <[email protected]>
yxieca pushed a commit that referenced this pull request Jan 30, 2020
…1158)

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

Signed-off-by: Stepan Blyschak <[email protected]>
lguohan pushed a commit that referenced this pull request Jan 30, 2020
…1158)

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

Signed-off-by: Stepan Blyschak <[email protected]>
abdosi pushed a commit that referenced this pull request Feb 4, 2020
…1158)

Otherwise, we can see that DUT after warm reboot sends ARP
packets with management IP (10.112.71.4)

Signed-off-by: Stepan Blyschak <[email protected]>
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
- What I did
Add an option to force the hardware register dump with -f for show techsupport --allow-process-stop
- How I did it
Add an option '--allow-process-stop' to show techsupport cli command and '-a' to generate_dump script
- How to verify it
Perform show techsupport --allow-process-stop to collect extra hardware registers information which may require system interruption.
- Previous command output (if the output of a command-line utility has changed)
N/A
- New command output (if the output of a command-line utility has changed)
N/A
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.

7 participants