Skip to content

Conversation

@vvoland
Copy link
Collaborator

@vvoland vvoland commented Oct 31, 2025

Sort images alphabetically by their repository tags rather than by creation date.

When an image has multiple tags, they are sorted internally and the first tag is used as the representative for sorting the image in the list. Untagged images are placed at the end.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

`docker image ls --tree` now sorts images alphabetically by name instead of by creation date

- A picture of a cute animal (not mandatory but encouraged)

Sort images alphabetically by their repository tags rather than by
creation date.

When an image has multiple tags, they are sorted internally and the
first tag is used as the representative for sorting the image in the
list. Untagged images are placed at the end.

Signed-off-by: Paweł Gronowski <[email protected]>
@vvoland vvoland added this to the 29.0.0 milestone Oct 31, 2025
@vvoland vvoland self-assigned this Oct 31, 2025
@vvoland vvoland added kind/enhancement area/ux containerd-integration Issues and PRs related to containerd integration area/images labels Oct 31, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/image/tree.go 0.00% 16 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah
Copy link
Member

Heh, still slightly confusing with multiple registries involved for the same image; looks like we may not be sorting the "per-group" list, and probably 127.0.0.1 sorted before alpine, but it's not the first one in the list of names we show 🤔

Screenshot 2025-10-31 at 16 52 06

@thaJeztah
Copy link
Member

Still; it's better than the old situation, but probably something we can follow-up on.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah merged commit 918ec8c into docker:master Oct 31, 2025
113 of 121 checks passed
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.

4 participants