feat: Add shared storage and lifecycle coordination - #17
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
WalkthroughThe bridge adds in-memory and protocol-based storage for node configs, queued user synchronization, and lifecycle leases. Controllers use leased claims and coordinated updates, REST/gRPC nodes report lifecycle states, and public helpers support registry-backed node creation. ChangesShared storage and lifecycle coordination
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant Controller
participant UserSyncStore
participant NodeLifecycleCoordinator
participant NodeAPI
Controller->>UserSyncStore: claim pending users
Controller->>NodeLifecycleCoordinator: acquire update lease
Controller->>NodeAPI: send coordinated update
NodeAPI-->>Controller: return update result
Controller->>UserSyncStore: acknowledge or requeue users
Controller->>NodeLifecycleCoordinator: release update lease
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
* feat: Add shared storage and lifecycle coordination * feat: Enhance lifecycle coordination with exclusive update operations and default stores
Summary by CodeRabbit
New Features
Documentation
Bug Fixes