Skip to content

Comments

ci: install scapy-rpc on Packit#4930

Merged
gpotter2 merged 1 commit intosecdev:masterfrom
evverx:scapy-rpc
Feb 23, 2026
Merged

ci: install scapy-rpc on Packit#4930
gpotter2 merged 1 commit intosecdev:masterfrom
evverx:scapy-rpc

Conversation

@evverx
Copy link
Contributor

@evverx evverx commented Feb 23, 2026

to prevent the tests from failing with

 ###(000)=[failed] [C706 MGMT] - Import layer
 ...
 ModuleNotFoundError: No module named 'scapy.layers.msrpce.raw.mgmt'

 ###(001)=[failed] [C706 MGMT] - Dissect rpc__mgmt_inq_if_ids_Response
 ...
 >>> pkt = rpc__mgmt_inq_if_ids_Response(DATA)
 NameError: name 'rpc__mgmt_inq_if_ids_Response' is not defined

scapy-rpc isn't packaged so it has to be installed using pip instead of Build-Requires as usual.

It's a follow-up to #4921.

It was tested in https://copr.fedorainfracloud.org/coprs/packit/evverx-scapy-2/build/10159677/.

to prevent the tests from failing with
```
 ###(000)=[failed] [C706 MGMT] - Import layer
 ...
 ModuleNotFoundError: No module named 'scapy.layers.msrpce.raw.mgmt'

 ###(001)=[failed] [C706 MGMT] - Dissect rpc__mgmt_inq_if_ids_Response
 ...
 >>> pkt = rpc__mgmt_inq_if_ids_Response(DATA)
 NameError: name 'rpc__mgmt_inq_if_ids_Response' is not defined
```

scapy-rpc isn't packaged so it has to be installed using pip instead of
Build-Requires as usual.

It's a follow-up to secdev#4921.
@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.69%. Comparing base (c9c91ba) to head (b722633).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4930   +/-   ##
=======================================
  Coverage   80.69%   80.69%           
=======================================
  Files         370      370           
  Lines       91759    91759           
=======================================
+ Hits        74041    74044    +3     
+ Misses      17718    17715    -3     

see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gpotter2
Copy link
Member

Thanks a lot !

@gpotter2 gpotter2 merged commit 5c3fe1c into secdev:master Feb 23, 2026
23 of 24 checks passed
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