Skip to content

chore: bump mem limit and remove progress for db backup#423

Merged
adibarra merged 2 commits into
masterfrom
chore/db-backup-remove-progress
Jun 5, 2026
Merged

chore: bump mem limit and remove progress for db backup#423
adibarra merged 2 commits into
masterfrom
chore/db-backup-remove-progress

Conversation

@adibarra
Copy link
Copy Markdown
Contributor

@adibarra adibarra commented Jun 5, 2026

Note

Low Risk
CI-only workflow changes with no app or data-path logic; slightly higher runner memory use during compression is the only operational tradeoff.

Overview
Simplifies the weekly Database Dump GitHub Actions workflow by dropping the 30-second background progress loggers on the dump and compress steps (and their trap cleanup).

Raises xz compression memory from --memlimit-compress=95% to 99% on the same lzma2=preset=9e,dict=192MiB pipeline before splitting into ~1.9 GiB parts for release upload.

Reviewed by Cursor Bugbot for commit e21c7a3. Bugbot is set up for automated code reviews on this repo. Configure here.

adibarra added 2 commits June 5, 2026 18:04
Remove the [dump]/[compress] elapsed-time heartbeat watchers; they added
log noise for little value. xz -v still prints a one-line end summary.
Lifts the multithreaded memory cap so xz runs 30 threads (was 29 at 95%),
leaving ~2 GiB for the OS on the ~118 GiB runner.
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview, Comment Jun 5, 2026 11:29pm

Request Review

@adibarra adibarra changed the title chore: db backup remove progress chore: bump mem limit and remove progress for db backup Jun 5, 2026
@adibarra adibarra marked this pull request as ready for review June 5, 2026 23:30
@adibarra adibarra merged commit 307e575 into master Jun 5, 2026
14 checks passed
@adibarra adibarra deleted the chore/db-backup-remove-progress branch June 5, 2026 23:30
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