Skip to content

fix: recover stale luks device mapper#1364

Draft
lukasmetzner wants to merge 4 commits into
mainfrom
fix-luks-recover-stale-dev-mapper
Draft

fix: recover stale luks device mapper#1364
lukasmetzner wants to merge 4 commits into
mainfrom
fix-luks-recover-stale-dev-mapper

Conversation

@lukasmetzner
Copy link
Copy Markdown
Contributor

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

❌ Patch coverage is 51.92308% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.64%. Comparing base (f4f1120) to head (4ab707d).

Files with missing lines Patch % Lines
internal/volumes/cryptsetup.go 52.00% 24 Missing ⚠️
internal/volumes/resize.go 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1364      +/-   ##
==========================================
- Coverage   74.34%   71.64%   -2.70%     
==========================================
  Files          17       17              
  Lines        1520     1552      +32     
==========================================
- Hits         1130     1112      -18     
- Misses        301      352      +51     
+ Partials       89       88       -1     
Flag Coverage Δ
e2e-controller 55.60% <19.23%> (-4.47%) ⬇️
e2e-node 55.60% <19.23%> (-4.47%) ⬇️
integration 3.10% <0.00%> (-0.09%) ⬇️
unit 41.10% <32.69%> (+0.25%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@lukasmetzner
Copy link
Copy Markdown
Contributor Author

Debugging information provided by customer indicates there is an issue with luks device mappers not being closed properly. I could not yet reproduce this correctly, but this would address the issue.

@lukasmetzner
Copy link
Copy Markdown
Contributor Author

lukasmetzner commented May 13, 2026

As an addition, I properly introduced context management into the cryptsetup. Required a little refactor, but this is mostly just dragging along the context.

@lukasmetzner lukasmetzner force-pushed the fix-luks-recover-stale-dev-mapper branch from 23ee4c1 to 5ff574f Compare May 29, 2026 13:14
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