Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Embedded the overlay snapshotter #1340

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rbpdt
Copy link

@rbpdt rbpdt commented Aug 10, 2023

Proof of concept for the suggested change of adding a metastore opt to containerd's overlay snapshotter.

@rbpdt rbpdt force-pushed the refactor/snapshot-embedded-containerd branch 2 times, most recently from 7fe792d to dfdd884 Compare August 11, 2023 10:11
@rbpdt rbpdt force-pushed the refactor/snapshot-embedded-containerd branch 2 times, most recently from a416f42 to ca856f6 Compare August 11, 2023 16:52
Signed-off-by: Robbie Buxton <138501839+rbpdt@users.noreply.github.com>
@rbpdt rbpdt force-pushed the refactor/snapshot-embedded-containerd branch from ca856f6 to 209aaf5 Compare August 11, 2023 17:05
Signed-off-by: Robbie Buxton <138501839+rbpdt@users.noreply.github.com>
@rbpdt rbpdt force-pushed the refactor/snapshot-embedded-containerd branch from 58165c0 to 2897250 Compare August 11, 2023 17:14
Signed-off-by: Robbie Buxton <138501839+rbpdt@users.noreply.github.com>
@rbpdt rbpdt changed the title WIP [WIP] Embedded the overlay snapshotter Aug 11, 2023
@ktock
Copy link
Member

ktock commented Aug 14, 2023

Thanks for the patch. This seems to depend on the on-going patch on containerd (containerd/containerd#8945 ) so let me turn this into a draft PR as of now.

@ktock ktock marked this pull request as draft August 14, 2023 11:56
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.

None yet

2 participants