[hip] Incremental snapshots - #1823
Conversation
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
ludfjig
changed the title
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The HIP text has a few clarity/formatting issues (including an ambiguous “cap” definition) that should be tightened to make the proposal easier to review and implement consistently.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review tier: Lite
Findings: 3
New issues introduced by this change (3)
| Severity | Finding |
|---|---|
 |
proposals/0003-incremental-snapshots/README.md — Fix run-on sentence and wrap lines in Motivation paragraph |
|
proposals/0003-incremental-snapshots/README.md — Remove extra blank lines before Proposal heading |
|
proposals/0003-incremental-snapshots/README.md — Clarify what determines the snapshot mapping cap |
What changed in this PR
Adds HIP 0003 proposing an incremental snapshot format for Hyperlight sandboxes, aiming to reduce snapshot latency and memory usage by sharing immutable snapshot “blobs” across snapshot layers.
Changes:
- Introduces a new HIP describing an incremental, layered snapshot data model (layers, blobs, live ranges).
- Describes algorithms for taking/restoring snapshots with minimal remapping by diffing live ranges.
- Proposes an OCI-based on-disk representation using one file per blob and a v2 config format.
| File | Description |
|---|---|
| proposals/0003-incremental-snapshots/README.md | New HIP documenting the incremental snapshot design, restore/take flow, and on-disk representation. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
ludfjig
added
the
kind/enhancement
jsturtevant
left a comment
jsturtevant
left a comment
There was a problem hiding this comment.
Overall looks like a good approach, left a few comments on the details.
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
Signed-off-by: Ludvig Liljenberg <4257730+ludfjig@users.noreply.github.com>
|
Discussed at community meeting (on Mondays find link here). Notes are in https://hackmd.io/AhiY2H3fRniOTVBje_4aWg The general discussion direction was agreed upon, please read and post comments here otherwise we will have lazy consensus for merging on the Sept 28th (2 weeks). |
Rendered