feat: add agent enroll time in inventory loop#188
Conversation
Signed-off-by: Amber Xue <ambermingxin@nvidia.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (6)
✅ Files skipped from review due to trivial changes (5)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds enrolled_at metadata support: State API and constant, sqlite persistence and tests, enrollment records UTC timestamp, inventory sink propagates EnrolledAt, unenroll cleans the key, and Go toolchain bumps. ChangesEnrollment Time Tracking
Sequence DiagramsequenceDiagram
participant EnrollFlow as Enrollment Flow
participant State as Agent State
participant InventorySink as Inventory Sink
participant Backend as Backend Client
EnrollFlow->>State: SetEnrollmentTime(enrolledAt)
State->>State: Store enrolledAt in metadata (RFC3339Nano)
InventorySink->>State: GetEnrollmentTime()
State-->>InventorySink: enrolledAt timestamp
InventorySink->>InventorySink: Build NodeUpsertRequest with EnrolledAt
InventorySink->>Backend: UpsertNode(request with EnrolledAt)
Backend-->>InventorySink: Success
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~30 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
mukilsh
left a comment
There was a problem hiding this comment.
LGTM, just take a look at that one comment
Signed-off-by: Amber Xue <ambermingxin@nvidia.com>
Signed-off-by: Amber Xue <ambermingxin@nvidia.com>
Signed-off-by: Amber Xue <ambermingxin@nvidia.com>
Signed-off-by: Amber Xue <ambermingxin@nvidia.com>
Description
Checklist
Summary by CodeRabbit
New Features
Bug Fixes
Tests
Chores