Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

Sort app image ls by created #756

Merged
merged 1 commit into from
Nov 21, 2019

Conversation

zappy-shu
Copy link
Contributor

- What I did

Sorts apps by created date (if available) when running app image ls with the latest created apps coming first.

- How I did it

Before printing the images they are first sorted by checking the custom payload. If the bundles contain a docker custom payload that has a creation date then this is used to sort them. If only one bundle has a creation date then it is considered the newer one. A stable sort is used so that non-App bundles and tags do not get muddled.

- How to verify it

Build multiple App images. Run docker app image ls. The result should order the images by creation date.

- Description for the changelog

App images are sorted by creation date with the newest first when running app image ls

- A picture of a cute animal (not mandatory)

bird-2019-11-19

@zappy-shu zappy-shu changed the title Sort app image ls by created [WIP] Sort app image ls by created Nov 20, 2019
@zappy-shu zappy-shu force-pushed the APP-344-sort-images-by-creat-date branch 2 times, most recently from a8967cb to acc00b1 Compare November 21, 2019 12:32
@zappy-shu zappy-shu changed the title [WIP] Sort app image ls by created Sort app image ls by created Nov 21, 2019
Sorts apps by created date (if available) when running `app image ls`
with the latest created apps coming first.

Signed-off-by: Nick Adcock <nick.adcock@docker.com>
@zappy-shu zappy-shu force-pushed the APP-344-sort-images-by-creat-date branch from acc00b1 to 0a3e85a Compare November 21, 2019 13:33
Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

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

LGTM

@rumpl rumpl merged commit 9249440 into docker:master Nov 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants