-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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.
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
[Logs UI] Correctly round and update the log stream end timestamps #152387
[Logs UI] Correctly round and update the log stream end timestamps #152387
Conversation
This way it works with the visual editor.
@elasticmachine merge upstream |
π Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @weltenwort |
Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I tested locally the scenarios I found broken and everything works fine now, thanks for taking on this so quickly π
π All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
The code containing the bug doesn't exist yet in 8.7, so I'll abort the backport. |
β¦lastic#152387) Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
π Summary
This fixes the handling of the end timestamp in the log stream page query state machine.
closes #152363
closes #152337
π΅οΈββοΈ Review notes
now
as the end time value.after
transition is properly recognized by the visual editor: