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.

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

Fix formatting for event.duration #11675

Merged
merged 2 commits into from
Apr 8, 2019
Merged

Conversation

webmat
Copy link
Contributor

@webmat webmat commented Apr 5, 2019

Human readable not working well for us.

I don't think this is a blocker for 7.0.0. The workaround is simply to edit the Kibana index pattern's formatting options for event.duration.

But this should absolutely go to 7.0.1 if possible.

Note that this is a quick fix applied on a file generated by ECS. If we go forward with this fix, I will backport it to ECS

Human readable not working well here.
@webmat webmat requested review from a team as code owners April 5, 2019 18:58
@webmat webmat self-assigned this Apr 5, 2019
@webmat webmat added the ecs label Apr 5, 2019
@webmat
Copy link
Contributor Author

webmat commented Apr 5, 2019

Here's a visualization from the Packetbeat Dashboard prior to this PR:

before-11675

Here's the same visualization after this PR:

after-11675

I don't think it looks great, but it's much more useful than the current state of things.

@webmat webmat added the v7.0.1 label Apr 5, 2019
@webmat webmat changed the title Override current ECS formatting for event.duration Fix formatting for event.duration Apr 5, 2019
@webmat
Copy link
Contributor Author

webmat commented Apr 5, 2019

Note that I've built one of the Beats, performed its beat setup, ingested data and confirmed that the format takes effect properly.

@webmat webmat requested a review from ruflin April 5, 2019 20:33
@ruflin
Copy link
Member

ruflin commented Apr 8, 2019

The new graph is better then the old one but still not very nice. Also it lacks the unit.

I think this is a more general issue Kibana elastic/kibana#20374

I'm good with getting this into Beats for now but not sure if I would make it part of ECS directly as I would hope we could get overall a better solution long term.

At the same time I would not consider these changes to be breaking changes so we could add it to ECS and change it later.

@ruflin
Copy link
Member

ruflin commented Apr 8, 2019

jenkins, test this

@exekias exekias added the needs_backport PR is waiting to be backported to other branches. label Apr 8, 2019
@webmat webmat merged commit 529885e into elastic:master Apr 8, 2019
@@ -613,6 +613,8 @@
type: long
format: duration
input_format: nanoseconds
output_format: asMilliseconds
output_precision: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Just as a side note, we previously had some confusion around showing durations that are stored and declared as nanoseconds in a different format (as this also changes format in discover), see elastic/apm-server#1717

webmat added a commit to webmat/beats that referenced this pull request Apr 10, 2019
Human readable not working well for us.
webmat added a commit that referenced this pull request Apr 11, 2019
Human readable not working well for us.
webmat pushed a commit to webmat/beats that referenced this pull request Apr 11, 2019
webmat added a commit that referenced this pull request Apr 15, 2019
webmat pushed a commit to webmat/beats that referenced this pull request Apr 15, 2019
webmat added a commit that referenced this pull request Apr 16, 2019
Backport of PR #11766 (changelog for #11675) to 7.0 branch.
DStape pushed a commit to DStape/beats that referenced this pull request Aug 20, 2019
Human readable not working well for us.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ecs needs_backport PR is waiting to be backported to other branches. v7.0.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants