Skip to content

Demote unhealthy primary object store sooner - #38

Merged
BotCoder254 merged 1 commit into
mainfrom
feat/log-chunk-sections
Jul 16, 2026
Merged

Demote unhealthy primary object store sooner#38
BotCoder254 merged 1 commit into
mainfrom
feat/log-chunk-sections

Conversation

@BotCoder254

Copy link
Copy Markdown
Owner

Unifies storage routing around a shared primary-health cache so both server-side writes and presigned upload routing proactively avoid a known-down primary when a fallback exists. Primary failures now mark the cache unhealthy immediately (including startup bucket checks), and routing automatically returns to primary after a healthy probe.

ensure_buckets now verifies all configured stores, keeps MinIO auto-create behavior, adds reachability warnings for non-MinIO backends, and seeds failover state when the primary is unreachable at boot. Added tests for boot-time demotion affecting upload routing and for single-store behavior never demoting. Updated CLAUDE.md to document the new failover/promotion behavior.

Unifies storage routing around a shared primary-health cache so both server-side writes and presigned upload routing proactively avoid a known-down primary when a fallback exists. Primary failures now mark the cache unhealthy immediately (including startup bucket checks), and routing automatically returns to primary after a healthy probe.

`ensure_buckets` now verifies all configured stores, keeps MinIO auto-create behavior, adds reachability warnings for non-MinIO backends, and seeds failover state when the primary is unreachable at boot. Added tests for boot-time demotion affecting upload routing and for single-store behavior never demoting. Updated `CLAUDE.md` to document the new failover/promotion behavior.
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
overup Ready Ready Preview, Comment Jul 16, 2026 10:04pm

@BotCoder254
BotCoder254 merged commit b5792a6 into main Jul 16, 2026
2 checks passed
@netlify

netlify Bot commented Jul 16, 2026

Copy link
Copy Markdown

Deploy Preview for overup-app ready!

Name Link
🔨 Latest commit 56e2a7b
🔍 Latest deploy log https://app.netlify.com/projects/overup-app/deploys/6a5955565b7b6c0008e8b557
😎 Deploy Preview https://deploy-preview-38--overup-app.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@amazon-q-developer amazon-q-developer Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The storage failover implementation correctly unifies routing around a shared primary-health cache. Both server-side writes and presigned upload grants now proactively avoid a known-down primary, and the cache is properly seeded from boot bucket checks and real write failures. The implementation uses appropriate async patterns with tokio Mutex for cache coalescing, proper TTL handling, and maintains backward compatibility with single-store deployments. Test coverage validates boot-time demotion and single-store behavior.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.

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