Skip to content
This repository has been archived by the owner on Feb 24, 2024. It is now read-only.

Wrong sort order for many columns #71

Open
godfuture opened this issue May 18, 2018 · 7 comments
Open

Wrong sort order for many columns #71

godfuture opened this issue May 18, 2018 · 7 comments

Comments

@godfuture
Copy link

godfuture commented May 18, 2018

If you try to sort your image library or container list, you will notice that some of the columns get ordered incorrect. image create date, image virtual size, container up time....and others.

omv_wrong_sorting

@subzero79
Copy link

subzero79 commented May 18, 2018

Size sorting was fixed long time. I haven’t look at the others. What version is this ?

@godfuture
Copy link
Author

Client:
Version: 18.03.1-ce
OS/Arch: linux/amd64

Server:
Engine:
Version: 18.03.1-ce
Built: Thu Apr 26 07:15:24 2018
OS/Arch: linux/amd64

@subzero79
Copy link

And the plugin? Or Omv version ?

@godfuture
Copy link
Author

godfuture commented May 19, 2018

docker-gui 4.0.1 on OMV 4.1.6

@subzero79
Copy link

4.0.2 has the size column in images fixed a long time ago.
As for the STATUS omv provides a unixtimestamp column, that i believe it will translate into date then be sortable, but docker ps needs to report in epoch time, which i believe it doesn't. In that case you will see on the grid the date-time it was started but not as friendly as it is now like "Up 11 minutes"

If you read here http://timjrobinson.com/get-docker-container-start-time-in-unix-timestamp-seconds/

you can see he is fetching low level container info, that means will have to parse each container to get that info, which i am reluctant because it would slow down the grid rendering.

Two options:

  • disable sorting for status
  • custom column js definition that can parse those strings.

As for now i am disabling sorting for STATUS, until someone wants to contribute some code for this.

@raulfg3
Copy link

raulfg3 commented Sep 21, 2018

@subzero79
Copy link

Not really, unless docker outputs the date created in parseable format.
I can query each container individually to get the date but that would slow down the container grid, the info we see in the container grid comes from one api query.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants