Skip to content

[scapy] update scapy to 2.4.5 and patch it#67

Closed
stepanblyschak wants to merge 1 commit intomasterfrom
upgrade_scapy_with_fix
Closed

[scapy] update scapy to 2.4.5 and patch it#67
stepanblyschak wants to merge 1 commit intomasterfrom
upgrade_scapy_with_fix

Conversation

@stepanblyschak
Copy link
Owner

Signed-off-by: Stepan Blyschak stepanb@nvidia.com

Why I did it

Running warm-reboot in a loop for 500 times leads to this error on 318-th iteration:

Apr  2 15:56:27.346747 sonic INFO swss#/supervisord: restore_neighbors Traceback (most recent call last):
Apr  2 15:56:27.346747 sonic INFO swss#/supervisord: restore_neighbors   File "/usr/bin/restore_neighbors.py", line 24, in <module>
Apr  2 15:56:27.346747 sonic INFO swss#/supervisord: restore_neighbors     from scapy.all import conf, in6_getnsma, inet_pton, inet_ntop, in6_getnsmac, get_if_hwaddr, Ether, ARP, IPv6, ICMPv6ND_NS, ICMPv6NDOptSrcLLAddr
Apr  2 15:56:27.346795 sonic INFO swss#/supervisord: restore_neighbors   File "/usr/local/lib/python3.7/dist-packages/scapy/all.py", line 25, in <module>
Apr  2 15:56:27.346956 sonic INFO swss#/supervisord: restore_neighbors     from scapy.route import *
Apr  2 15:56:27.346995 sonic INFO swss#/supervisord: restore_neighbors   File "/usr/local/lib/python3.7/dist-packages/scapy/route.py", line 205, in <module>
Apr  2 15:56:27.347089 sonic INFO swss#/supervisord: restore_neighbors     conf.iface = get_working_if()
Apr  2 15:56:27.347129 sonic INFO swss#/supervisord: restore_neighbors   File "/usr/local/lib/python3.7/dist-packages/scapy/arch/linux.py", line 128, in get_working_if
Apr  2 15:56:27.347213 sonic INFO swss#/supervisord: restore_neighbors     ifflags = struct.unpack("16xH14x", get_if(i, SIOCGIFFLAGS))[0]
Apr  2 15:56:27.347250 sonic INFO swss#/supervisord: restore_neighbors   File "/usr/local/lib/python3.7/dist-packages/scapy/arch/common.py", line 31, in get_if
Apr  2 15:56:27.347345 sonic INFO swss#/supervisord: restore_neighbors     return ioctl(sck, cmd, struct.pack("16s16x", iff.encode("utf8")))
Apr  2 15:56:27.347365 sonic INFO swss#/supervisord: restore_neighbors OSError: [Errno 19] No such device

The issue was reported to scapy devs secdev/scapy#3369, the fix is secdev/scapy#3371, however there is no released scapy version with this fix right now, thus decided to build scapy v2.4.5 from sources and apply the fix in a form of a patch.

How I did it

Added scapy as a submodule and applied the fix.

How to verify it

Run warm-reboot in a loop

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@stepanblyschak stepanblyschak force-pushed the upgrade_scapy_with_fix branch 2 times, most recently from caa267c to 2812668 Compare April 4, 2022 10:21
Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
@stepanblyschak stepanblyschak force-pushed the upgrade_scapy_with_fix branch from 2812668 to 65d7a85 Compare April 4, 2022 11:04
stepanblyschak pushed a commit that referenced this pull request May 26, 2022
[sonic-linkmgrd][202012] submodule update
3d13ff2 Jing Zhang      Wed May 4 10:07:14 2022 -0700   Add doc for default route related changes  (#63)
c703be4 Jing Zhang      Mon May 2 13:27:54 2022 -0700   Reset WaitActiveUp count before switching to active (#70)
86eb727 Jing Zhang      Wed Apr 27 10:35:05 2022 -0700  lower log level to warning (#69)
e22c736 Jing Zhang      Mon May 2 13:33:24 2022 -0700   [202012] Avoid proactively switching to active if default route is missing (#67)
d4f282b Jing Zhang      Thu Apr 28 18:35:11 2022 -0700  [202012] Add support to enable switchover time measurement (with link prober interval decreased to 10ms) feature (#66)

sign-off: Jing Zhang [zhangjing@microsoft.com](mailto:zhangjing@microsoft.com)
@stepanblyschak stepanblyschak deleted the upgrade_scapy_with_fix branch September 23, 2022 13:34
stepanblyschak pushed a commit that referenced this pull request Jun 30, 2023
…utomatically (sonic-net#15548)

#### Why I did it
src/sonic-host-services
```
* 508d642 - (HEAD -> master, origin/master, origin/HEAD) [actions] Support Semgrep by Github Actions (#67) (31 hours ago) [Mai Bui]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit that referenced this pull request May 20, 2025
…tomatically (sonic-net#992)

#### Why I did it
src/sonic-swss
```
* 7498ae9 - (HEAD -> 202412, origin/202412) Merge pull request #67 from mssonicbld/sonicbld/202412-merge (21 hours ago) [mssonicbld]
* bbce4b4 - Merge branch '202411' of https://github.com/sonic-net/sonic-swss into 202412 (21 hours ago) [Sonic Automation]
* fceb196 - Revert "Optimize counter polling interval by making it more accurate (11 days ago) [Kumaresh Perumal]
* d88b694 - Remove unused string variables in flexcounterorch.cpp (12 days ago) [Kumaresh Perumal]
|\ 
| failure_prs.log skip_prs.log 1e4315e - Remove unused string variables in flexcounterorch.cpp (12 days ago) [Kumaresh Perumal]
|/ 
* e82f8c2 - Revert "Optimize counter polling interval by making it more accurate (sonic-net#3500)" (12 days ago) [Kumaresh Perumal]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit that referenced this pull request Jul 22, 2025
…ly (sonic-net#23161)

#### Why I did it
src/sonic-ztp
```
* f8fa833 - (HEAD -> master, origin/master, origin/HEAD) [ci] Upgrade agent pool from ubuntu 20.04 to latest. (#67) (22 hours ago) [Liu Shilong]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit that referenced this pull request Jul 22, 2025
…ically (sonic-net#23211)

#### Why I did it
src/sonic-dash-ha
```
* ff1b02a - (HEAD -> master, origin/master, origin/HEAD) Implement HA-Scope actor (#73) (20 hours ago) [yue-fred-gao]
* 832e67a - Implement HA-SET actor (#70) (4 days ago) [yue-fred-gao]
* cadfa46 - Implement vdpu actor (#67) (5 days ago) [yue-fred-gao]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit that referenced this pull request Dec 1, 2025
…ly (sonic-net#24445)

#### Why I did it
src/dhcprelay
```
* 8772dd5 - (HEAD -> master, origin/master, origin/HEAD) [dhcp_relay] sonic dhcp relay agent for IPv4 (#67) (20 hours ago) [Shivashankar C R]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants