Skip to content

Fix failing fedora distro test - #2908

Merged
liquidsec merged 7 commits into
devfrom
fix-fedora-test
Feb 19, 2026
Merged

Fix failing fedora distro test#2908
liquidsec merged 7 commits into
devfrom
fix-fedora-test

Conversation

@liquidsec

Copy link
Copy Markdown
Collaborator

pyenv needs patch to build Python from source, but it wasn't included in the Fedora dnf install list, causing the build to fail.

pyenv needs `patch` to build Python from source, but it wasn't
included in the Fedora dnf install list, causing the build to fail.
@codecov

codecov Bot commented Feb 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92%. Comparing base (ac39c2b) to head (b60e7ae).
⚠️ Report is 8 commits behind head on dev.

Additional details and impacted files
@@          Coverage Diff          @@
##             dev   #2908   +/-   ##
=====================================
+ Coverage     92%     92%   +1%     
=====================================
  Files        416     416           
  Lines      34398   34399    +1     
=====================================
+ Hits       31356   31371   +15     
+ Misses      3042    3028   -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

Fedora 43 replaces p7zip with native 7zip (v25.01), which rejects the
literal -p"" argument passed via subprocess (exit code 2). The flag is
unnecessary for unencrypted archives.
@github-actions

github-actions Bot commented Feb 18, 2026

Copy link
Copy Markdown
Contributor

📊 Performance Benchmark Report

Comparing dev (baseline) vs fix-fedora-test (current)

📈 Detailed Results (All Benchmarks)

📋 Complete results for all benchmarks - includes both significant and insignificant changes

🧪 Test Name 📏 Base 📏 Current 📈 Change 🎯 Status
Bloom Filter Dns Mutation Tracking Performance 4.35ms 4.27ms -1.9%
Bloom Filter Large Scale Dns Brute Force 18.04ms 17.94ms -0.5%
Large Closest Match Lookup 360.66ms 347.42ms -3.7%
Realistic Closest Match Workload 195.02ms 193.59ms -0.7%
Event Validation Full Scan Startup Small Batch 493.30ms 482.12ms -2.3%
Event Validation Full Scan Startup Large Batch 843.14ms 821.67ms -2.5%
Make Event Autodetection Small 31.45ms 30.93ms -1.7%
Make Event Autodetection Large 314.02ms 316.07ms +0.7%
Make Event Explicit Types 13.78ms 14.43ms +4.7%
Excavate Single Thread Small 4.169s 4.243s +1.8%
Excavate Single Thread Large 10.014s 9.858s -1.6%
Excavate Parallel Tasks Small 4.390s 4.228s -3.7%
Excavate Parallel Tasks Large 7.418s 7.295s -1.7%
Is Ip Performance 3.18ms 3.24ms +1.9%
Make Ip Type Performance 11.60ms 11.55ms -0.5%
Mixed Ip Operations 4.53ms 4.51ms -0.4%
Typical Queue Shuffle 62.22µs 61.66µs -0.9%
Priority Queue Shuffle 708.39µs 692.17µs -2.3%

🎯 Performance Summary

No significant performance changes detected (all changes <10%)


🐍 Python Version 3.11.14

@liquidsec

liquidsec commented Feb 18, 2026

Copy link
Copy Markdown
Collaborator Author

On Fedora 43, /usr/sbin is a symlink to /usr/bin. When shutil.which("7z") resolves the executable, it may return /usr/sbin/7z. Native 7-Zip relies on argv[0] to locate its codec module (7z.so). When launched via the symlinked path, codec discovery fails, producing:

Codec Load Error: errno=25

All archive extractions then fail with exit code 2.

Fix:

Normalize the executable path by resolving directory symlinks only so the canonical path (/usr/bin/7z) is used. Preserve the binary name itself to avoid breaking virtualenv shims or intentional executable symlinks.

@liquidsec
liquidsec merged commit 0fd43c6 into dev Feb 19, 2026
23 of 24 checks passed
@liquidsec
liquidsec deleted the fix-fedora-test branch February 19, 2026 02:39
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