Skip to content

Sh 3985 Daemon Cleanup Pulled Image feature flag (disabled by default)#417

Open
houdini91 wants to merge 4 commits into
anchore:mainfrom
scribe-security:SH-3985-cleanup-images_2
Open

Sh 3985 Daemon Cleanup Pulled Image feature flag (disabled by default)#417
houdini91 wants to merge 4 commits into
anchore:mainfrom
scribe-security:SH-3985-cleanup-images_2

Conversation

@houdini91

Copy link
Copy Markdown

Second Suggestion: Cleanup Only when active pull by provider.
See anchore/syft#3985

Comment thread pkg/image/containerd/daemon_provider.go Outdated
Signed-off-by: houdini91 <mdstrauss91@gmail.com>
@houdini91 houdini91 force-pushed the SH-3985-cleanup-images_2 branch from 4236a4c to 7acfcd5 Compare June 25, 2025 19:27
Signed-off-by: houdini91 <mdstrauss91@gmail.com>
@kzantow

kzantow commented Jun 26, 2025

Copy link
Copy Markdown
Contributor

Hey @houdini91 just noting here we will want to make sure this is disabled by default and opt in; I noted in the original issue.

Signed-off-by: houdini91 <mdstrauss91@gmail.com>
@houdini91 houdini91 marked this pull request as ready for review June 30, 2025 06:51
@houdini91

Copy link
Copy Markdown
Author

@kzantow I've updated the code to reflect your comment. If you prefer a different approach, please let me know. (One unexplored option in this PR was using an environment variable like TMPDIR, perhaps IMAGE_CLEANUP_ENABLE, to keep provider constructors unchanged.)"

Signed-off-by: houdini91 <mdstrauss91@gmail.com>
@houdini91

Copy link
Copy Markdown
Author

@kzantow Fixed Lint error

@houdini91 houdini91 changed the title Sh 3985 cleanup images 2 Sh 3985 Daemon Cleanup Pulled Image feature flag (disabled by default) Jul 14, 2025
@dn-scribe

Copy link
Copy Markdown

A needed feature when running on CIs or when running to generate sboms at scale

@houdini91

Copy link
Copy Markdown
Author

Up

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.

3 participants