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

Use Kibana byte formatter with ECS bytes fields #10184

Merged
merged 1 commit into from
Jan 24, 2019

Conversation

andrewkroh
Copy link
Member

Add a Kibana field formatter to the fields that contain bytes so that they are represented in human readable format (e.g. 10 KB, 10GB).

@andrewkroh andrewkroh requested review from a team as code owners January 18, 2019 19:06
@andrewkroh andrewkroh force-pushed the feature/ecs-byte-formatter branch from 0795ea5 to 7dbcfbb Compare January 18, 2019 19:06
Copy link
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Could we port this also to the ECS repo?

@@ -11,8 +11,8 @@ stop:


up:
${BASE_COMMAND} build
${BASE_COMMAND} up
${BASE_COMMAND} build elasticsearch kibana
Copy link
Contributor

Choose a reason for hiding this comment

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

leftover?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, thanks for catching this. I have undone this change.

Copy link
Contributor

Choose a reason for hiding this comment

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

Probably not pushed yet? Will also need an other rebase.

Copy link
Member Author

Choose a reason for hiding this comment

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

Pushed now. Was about to push then I realized it also needed a rebase then I got interrupted by a flight.

@andrewkroh andrewkroh force-pushed the feature/ecs-byte-formatter branch 2 times, most recently from d0ff9de to 0cde7d6 Compare January 23, 2019 16:53
@andrewkroh
Copy link
Member Author

An issue has be opened for ECS: elastic/ecs#305

@andrewkroh andrewkroh force-pushed the feature/ecs-byte-formatter branch from 0cde7d6 to f3f6e51 Compare January 24, 2019 16:55
Add a Kibana field formatter to the fields that contain bytes so that they are represented in human readable format (e.g. 10 KB, 10GB).
@andrewkroh andrewkroh force-pushed the feature/ecs-byte-formatter branch from f3f6e51 to ed1461a Compare January 24, 2019 17:41
@andrewkroh andrewkroh merged commit bfc2f32 into elastic:master Jan 24, 2019
DStape pushed a commit to DStape/beats that referenced this pull request Aug 20, 2019
Add a Kibana field formatter to the fields that contain bytes so that they are represented in human readable format (e.g. 10 KB, 10GB).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants