Skip to content

test: add unshare crash test case#486

Draft
bdrung wants to merge 2 commits intocanonical:mainfrom
bdrung:unshare
Draft

test: add unshare crash test case#486
bdrung wants to merge 2 commits intocanonical:mainfrom
bdrung:unshare

Conversation

@bdrung
Copy link
Member

@bdrung bdrung commented Mar 27, 2025

Add an unshare crash test case that is a stripped down version of a crash inside a container. This test case add code coverage to the FileNotFoundError exception code path in forward_crash_to_container. See also LP #2080499.

This merge request depends on #484.

@codecov
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

Attention: Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.

Project coverage is 29.08%. Comparing base (f713fe1) to head (da5f493).

Files with missing lines Patch % Lines
data/apport 0.00% 11 Missing ⚠️
tests/integration/test_signal_crashes.py 0.00% 10 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (f713fe1) and HEAD (da5f493). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (f713fe1) HEAD (da5f493)
4 1
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #486       +/-   ##
===========================================
- Coverage   84.21%   29.08%   -55.14%     
===========================================
  Files         104       88       -16     
  Lines       20778    19506     -1272     
  Branches     3253     2924      -329     
===========================================
- Hits        17498     5673    -11825     
- Misses       2827    13338    +10511     
- Partials      453      495       +42     

☔ 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.

@bdrung bdrung force-pushed the unshare branch 3 times, most recently from 8c37db5 to fdaf3c1 Compare April 11, 2025 17:48
@bdrung bdrung force-pushed the unshare branch 2 times, most recently from befa816 to 13ed286 Compare May 9, 2025 15:36
@bdrung bdrung force-pushed the unshare branch 2 times, most recently from 9d2cd06 to 7cb51fb Compare May 30, 2025 19:54
@bdrung bdrung added this to the 2.33.0 milestone May 30, 2025
@bdrung
Copy link
Member Author

bdrung commented Jun 3, 2025

Running the unshare test case in a privileged Docker container works except one issue: The namespace is the same.

I'll leave it to the reader to figure out how to make it work in the container.

@bdrung bdrung removed this from the 2.33.0 milestone Jun 3, 2025
@bdrung bdrung marked this pull request as draft June 3, 2025 12:46
bdrung added 2 commits June 6, 2025 14:22
Add an unshare crash test case that is a stripped down version of a
crash inside a container. This test case add code coverage to the
`FileNotFoundError` exception code path in `forward_crash_to_container`.
See also LP #2080499.
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.

1 participant