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.

Sign up for GitHub

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

Jump to bottom

up logs sort does not factor in am / pm for sort order #715

Open
4 tasks done
sanjaypojo opened this issue Oct 4, 2018 · 3 comments
Open
4 tasks done

up logs sort does not factor in am / pm for sort order #715

sanjaypojo opened this issue Oct 4, 2018 · 3 comments
Labels
bug

Comments

@sanjaypojo
Copy link

sanjaypojo commented Oct 4, 2018 •
edited
Loading

Prerequisites

  • I am running the latest version. (up upgrade)
  • I searched to see if the issue already exists.
  • I inspected the verbose debug output with the -v, --verbose flag.
  • Are you an Up Pro subscriber?

Description

up logs prints logs sorted by time such that the am logs show up after the pm logs (should be the other way)

Steps to Reproduce

Run up logs in the evening.
@sanjaypojo sanjaypojo changed the title up logs sort does not factor in am / pm up logs sort does not factor in am / pm for sort Oct 4, 2018
@sanjaypojo sanjaypojo changed the title up logs sort does not factor in am / pm for sort up logs sort does not factor in am / pm for sort order Oct 4, 2018
@tj
Copy link
Member

tj commented Oct 4, 2018

oh weird! I think I'm just printing them as they're received from AWS, should be ascending but I'll see if I can reproduce

@tj
Copy link
Member

tj commented Oct 4, 2018 •
edited
Loading

looks correct in mine, possibly a race condition though I'll dig some more

@sanjaypojo
Copy link
Author

Yeah, I didn't have this issue before, but it's happening to me now:
image

@tj tj added the bug label Oct 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tj @sanjaypojo