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

Filter dangling images using image filters. #1785

Merged
merged 1 commit into from
Dec 31, 2022

Conversation

manugupt1
Copy link
Contributor

Signed-off-by: Manu Gupta manugupt1@gmail.com

@manugupt1 manugupt1 added this to the v1.2.0 milestone Dec 31, 2022
@manugupt1 manugupt1 added the enhancement New feature or request label Dec 31, 2022
pkg/imgutil/imgutil.go Outdated Show resolved Hide resolved
pkg/imgutil/imgutil.go Outdated Show resolved Hide resolved
@@ -369,7 +370,7 @@ func ParseRepoTag(imgName string) (string, string) {

ref, err := refdocker.ParseDockerRef(imgName)
if err != nil {
logrus.WithError(err).Warnf("unparsable image name %q", imgName)
logrus.WithError(err).Debugf("unparsable image name %q", imgName)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Contributor Author

@manugupt1 manugupt1 Dec 31, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every time there is a <none> image listed out; this Warning is printed out; I think we should not do it; now that we are adding more support for dangling images.

Signed-off-by: Manu Gupta <manugupt1@gmail.com>
@manugupt1 manugupt1 marked this pull request as ready for review December 31, 2022 06:59
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@AkihiroSuda AkihiroSuda merged commit d7cf13c into containerd:main Dec 31, 2022
@manugupt1 manugupt1 deleted the filter-dangling-images branch March 31, 2023 05:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants