Skip to content

remove bbolt integrity check entirely (release/v2) - #116

Merged
DQSevilla merged 2 commits into
release/v2from
devin/1784146784-bbolt-sigterm-v2
Jul 15, 2026
Merged

remove bbolt integrity check entirely (release/v2)#116
DQSevilla merged 2 commits into
release/v2from
devin/1784146784-bbolt-sigterm-v2

Conversation

@Trent-TSE

@Trent-TSE Trent-TSE commented Jul 15, 2026

Copy link
Copy Markdown

Summary

release/v2 counterpart of #115: drops the bbolt integrity check entirely per discussion.

The check's RuntimeMaxSec=6s cap SIGTERMs (exit 143) bbolt check on large cache.db files, which was misread as corruption and skipped the sticky disk commit on every run, freezing the builder cache. The check was added for the copy-up corruption bug fixed in BLA-3252 and no longer fires legitimately.

  • delete checkBoltDbIntegrity and its commit-gating in the post step
  • delete reporter.reportIntegrityCheckFailure
  • skip-integrity-check input kept in action.yml (marked deprecated, no effect)
  • post-step logDatabaseHashes retained, relabeled "before unmount"

dist/ rebuilt with pnpm build.

Link to Devin session: https://app.devin.ai/sessions/35c21220e53a4e43a8287135eba4351d
Requested by: @Trent-TSE


View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

… skip

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration devin-ai-integration Bot changed the title fix: don't treat RuntimeMaxSec SIGTERM (exit 143) as bbolt corruption (release/v2) remove bbolt integrity check entirely (release/v2) Jul 15, 2026
@DQSevilla
DQSevilla merged commit 7a99e9f into release/v2 Jul 15, 2026
8 checks passed
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.

3 participants