Skip to content

revert containerd changes#297

Merged
jabrown85 merged 6 commits intomainfrom
jab/revert-containerd-changes
Sep 9, 2025
Merged

revert containerd changes#297
jabrown85 merged 6 commits intomainfrom
jab/revert-containerd-changes

Conversation

@jabrown85
Copy link
Copy Markdown
Contributor

Reverting all of this for now - we can probably bring some of this back but I am not sure what bits are busted and want to bring mainline back to safe.

@jabrown85 jabrown85 self-assigned this Sep 8, 2025
@jabrown85 jabrown85 requested a review from a team as a code owner September 8, 2025 18:53
@jabrown85 jabrown85 merged commit 9db16db into main Sep 9, 2025
2 of 3 checks passed
jjbustamante added a commit to buildpacks/pack that referenced this pull request Nov 27, 2025
Upgrades imgutil from v0.0.0-20250814164739-4b1c8875ba7e to
v0.0.0-20250909162057-9db16db815e3 to fix an issue where builder
images created with Docker's containerd snapshotter storage driver
had malformed manifests.

The bug caused the first several base image layers to be replaced
with empty blob references in the manifest, while the config
retained correct diff_ids. This created invalid images that failed
OCI spec validation with tools like skopeo.

The fix was already merged in buildpacks/imgutil#297, which reverted
the problematic "fast path" optimization commits.

Resolves #2490

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
jjbustamante added a commit to buildpacks/pack that referenced this pull request Nov 27, 2025
Upgrades imgutil from v0.0.0-20250814164739-4b1c8875ba7e to
v0.0.0-20250909162057-9db16db815e3 to fix an issue where builder
images created with Docker's containerd snapshotter storage driver
had malformed manifests.

The bug caused the first several base image layers to be replaced
with empty blob references in the manifest, while the config
retained correct diff_ids. This created invalid images that failed
OCI spec validation with tools like skopeo.

The fix was already merged in buildpacks/imgutil#297, which reverted
the problematic "fast path" optimization commits.

Resolves #2490

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Juan Bustamante <bustamantejj@gmail.com>
jjbustamante added a commit to buildpacks/pack that referenced this pull request Nov 27, 2025
Upgrades imgutil from v0.0.0-20250814164739-4b1c8875ba7e to
v0.0.0-20250909162057-9db16db815e3 to fix an issue where builder
images created with Docker's containerd snapshotter storage driver
had malformed manifests.

The bug caused the first several base image layers to be replaced
with empty blob references in the manifest, while the config
retained correct diff_ids. This created invalid images that failed
OCI spec validation with tools like skopeo.

The fix was already merged in buildpacks/imgutil#297, which reverted
the problematic "fast path" optimization commits.

Resolves #2490

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: Juan Bustamante <bustamantejj@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
@edmorley
Copy link
Copy Markdown
Contributor

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.

4 participants