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

fix(cleanup): fix cleanup for Buildah images by switching to tag-based logic #6437

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

alexey-igrychev
Copy link
Member

@alexey-igrychev alexey-igrychev commented Nov 22, 2024

The cleanup process for images built using Buildah was not working as expected due to an issue with digest-based parent lookup, as outlined in Buildah #3866. This issue occurs because Buildah currently does not provide a reliable way to retrieve the digest of an image without publishing it to a registry.

To address this, the cleanup logic has been updated to use image tags instead of digests as the unique identifier for the images. This change ensures that the cleanup process works as expected, even for images built with Buildah.

fix #6373

Signed-off-by: Aleksei Igrychev <[email protected]>
@alexey-igrychev alexey-igrychev force-pushed the fix/cleanup/support-images-built-with-buildah branch from 28dac2f to d5df5a4 Compare November 22, 2024 13:36
@alexey-igrychev alexey-igrychev changed the title fix(cleanup): replace digest-based cleanup with tag-based logic fix(cleanup): replace digest-based parent lookup with tag-based logic Nov 22, 2024
@alexey-igrychev alexey-igrychev force-pushed the fix/cleanup/support-images-built-with-buildah branch 2 times, most recently from f8fadc3 to 8e88661 Compare November 22, 2024 13:45
@alexey-igrychev alexey-igrychev changed the title fix(cleanup): replace digest-based parent lookup with tag-based logic @alexey-igrychev fix(cleanup): fix cleanup for Buildah images by switching to tag-based logic Nov 22, 2024
@alexey-igrychev alexey-igrychev changed the title @alexey-igrychev fix(cleanup): fix cleanup for Buildah images by switching to tag-based logic fix(cleanup): fix cleanup for Buildah images by switching to tag-based logic Nov 22, 2024
@alexey-igrychev alexey-igrychev force-pushed the fix/cleanup/support-images-built-with-buildah branch from 8e88661 to f7503fc Compare November 22, 2024 13:55
@alexey-igrychev alexey-igrychev marked this pull request as ready for review November 22, 2024 13:56
…d logic

The cleanup process for images built using Buildah was not working as expected due to an issue with digest-based parent lookup, as outlined in Buildah #3866. This issue occurs because Buildah currently does not provide a reliable way to retrieve the digest of an image without publishing it to a registry.

To address this, the cleanup logic has been updated to use image tags instead of digests as the unique identifier for the images. This change ensures that the cleanup process works as expected, even for images built with Buildah.

Signed-off-by: Aleksei Igrychev <[email protected]>
@alexey-igrychev alexey-igrychev force-pushed the fix/cleanup/support-images-built-with-buildah branch from dc41004 to f40381d Compare November 25, 2024 14:41
@alexey-igrychev alexey-igrychev merged commit 73fcc1b into main Nov 25, 2024
10 checks passed
@alexey-igrychev alexey-igrychev deleted the fix/cleanup/support-images-built-with-buildah branch November 25, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cleanup does not work for images built with Buildah
2 participants