diff options
| author | Valentin Rothberg <rothberg@redhat.com> | 2021-07-19 11:41:04 +0200 |
|---|---|---|
| committer | Valentin Rothberg <rothberg@redhat.com> | 2021-07-26 09:28:17 +0200 |
| commit | 1b6423e9f172b6d2437011ef4504a8247728a73d (patch) | |
| tree | 0ba7fccceb6b036ed5e49caec51ffeb9921234a8 /pkg/domain | |
| parent | ec5c7c1f6a1898dacddb6cc35802525c288b61ef (diff) | |
| download | podman-1b6423e9f172b6d2437011ef4504a8247728a73d.tar.gz podman-1b6423e9f172b6d2437011ef4504a8247728a73d.tar.bz2 podman-1b6423e9f172b6d2437011ef4504a8247728a73d.zip | |
refine dangling checks
By proxy by vendoring containers/common. Previously, a "dangling" image
was an untagged image; just a described in the Docker docs. The
definition of dangling has now been refined to an untagged image without
children to be compatible with Docker.
Further update a redundant image-prune test.
Fixes: #10998
Fixes: #10832
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'pkg/domain')
| -rw-r--r-- | pkg/domain/infra/abi/images_list.go | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/pkg/domain/infra/abi/images_list.go b/pkg/domain/infra/abi/images_list.go index b0e947991..2ec4ad244 100644 --- a/pkg/domain/infra/abi/images_list.go +++ b/pkg/domain/infra/abi/images_list.go @@ -30,12 +30,16 @@ func (ir *ImageEngine) List(ctx context.Context, opts entities.ImageListOptions) for j, d := range img.Digests() { digests[j] = string(d) } + isDangling, err := img.IsDangling(ctx) + if err != nil { + return nil, errors.Wrapf(err, "error checking if image %q is dangling", img.ID()) + } e := entities.ImageSummary{ ID: img.ID(), // ConfigDigest: string(img.ConfigDigest), Created: img.Created().Unix(), - Dangling: img.IsDangling(), + Dangling: isDangling, Digest: string(img.Digest()), RepoDigests: digests, History: img.NamesHistory(), |