feat(blog): add Compute time sync blog post#7984
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
4823fb7 to
bee02a1
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
WalkthroughAdds a new blog post with frontmatter and article text about Prisma Compute guest clock drift, the drawbacks of running NTP inside minimal microVMs, and using the host-paired KVM PTP clock to keep ChangesPrisma Compute time synchronization post
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
bee02a1 to
aadbd6e
Compare
aadbd6e to
af7d3bf
Compare
af7d3bf to
fc4dd23
Compare
fc4dd23 to
36b855d
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/blog/content/blog/prisma-compute-time-synchronization/index.mdx`:
- Around line 40-42: The consumer-hardware comparison in this prose is too broad
relative to the probe’s measured drift, so either add a citation for the 10x to
50x claim or soften it to a clearly qualified estimate. Update the wording in
the blog content around the drift discussion to keep the `13.75 ms/hour` result
clearly scoped to the experiment and avoid presenting the consumer-grade
comparison as an unqualified fact.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: be9f32cd-915e-4938-880b-012ce21b93ae
⛔ Files ignored due to path filters (2)
apps/blog/public/prisma-compute-time-synchronization/imgs/hero.svgis excluded by!**/*.svgapps/blog/public/prisma-compute-time-synchronization/imgs/meta.pngis excluded by!**/*.png
📒 Files selected for processing (1)
apps/blog/content/blog/prisma-compute-time-synchronization/index.mdx
ea2f69a to
4e06378
Compare
4e06378 to
47ea829
Compare
47ea829 to
e7baf6b
Compare
Write a blog post about time synchronization in long running Compute VMs. Signed-off-by: Alexey Orlenko's AI Agent <robot@aqrln.net>
e7baf6b to
dc885d1
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/blog/content/blog/prisma-compute-time-synchronization/index.mdx`:
- Around line 28-30: The `LoadSnapshot` paragraph in
`prisma-compute-time-synchronization` conflates `clock_realtime: true` with
`VMClock` support; update the wording to clearly separate these features. In the
relevant prose around `clock_realtime` and `VMClock`, state that
`clock_realtime: true` is x86_64-only and requires host Linux 5.16+, while
`VMClock` has its own partial support in recent Firecracker releases, limited to
snapshot-safety/restore-detection rather than broad clock drift handling. Keep
the feature/version matrix precise and avoid the current blanket statement about
restore detection support.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 822d60a2-0984-4247-8370-78e96140485b
⛔ Files ignored due to path filters (2)
apps/blog/public/prisma-compute-time-synchronization/imgs/hero.svgis excluded by!**/*.svgapps/blog/public/prisma-compute-time-synchronization/imgs/meta.pngis excluded by!**/*.png
📒 Files selected for processing (1)
apps/blog/content/blog/prisma-compute-time-synchronization/index.mdx
Write a blog post about time synchronization in long running Compute VMs.
Summary by CodeRabbit