Skip to content

Fix long-running backup resource handling - #755

Merged
itsDNNS merged 1 commit into
mainfrom
fix/long-running-backup-resources
Jul 28, 2026
Merged

Fix long-running backup resource handling#755
itsDNNS merged 1 commit into
mainfrom
fix/long-running-backup-resources

Conversation

@itsDNNS

@itsDNNS itsDNNS commented Jul 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • write scheduled and manual backup archives to disk-backed temporary files instead of buffering complete archives in memory
  • publish scheduled backups atomically and clean temporary files after failures or completed downloads
  • retain long-running collector futures until completion and prevent duplicate submissions, including during collector hot-swaps
  • add regression coverage for backup lifecycle and delayed collector completion

Verification

  • python -m pytest tests --ignore=tests/e2e -q: 3002 passed, 1 skipped
  • focused backup and collector tests: 84 passed
  • 64 MiB incompressible archive smoke test: peak RSS reduced from 156.0 MiB to 27.7 MiB with the same archive size
  • git diff --check and Python compilation passed

Fixes #738

@itsDNNS
itsDNNS merged commit a5e7e7b into main Jul 28, 2026
14 checks passed
@itsDNNS
itsDNNS deleted the fix/long-running-backup-resources branch July 28, 2026 09:30
@itsDNNS itsDNNS mentioned this pull request Jul 28, 2026
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.

Memory leak?

1 participant