-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Infrastructure UI] Add network in/out to Hosts View Table #139316
Comments
Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI) |
hey @elastic/kibana-vis-editors - as far as a workaround for the linked issue: once ad hoc data views is merged (expected for 8.5)...I believe they can work around this by using an Ad Hoc data view (by value) and a field formatter leveraging numeral.js to get that pattern. Can you confirm? edit: I played a bit and realized we don't allow the suffix at the same time as the "Default" numeral.js patterns. will discuss with the team |
Note: here is how Infra UI handles bit formatting: |
We could add a new entry to the list of Lens format overrides for bit formatting: As the underlying numeral.js formatting already supports this, it should be easy to add. What do you think @ghudgins ? |
@ghudgins Is it possible this could be available for us to use before the 8.6 FF? :) |
we're doing planning this week but we do have this on the list for 8.6 - #139639 we'll make sure it happens as early as we can before FF |
Looks like #141372 unblocks this. |
Let us know in case something isn’t working right - still very easy to do changes at this point |
@flash1293 When using the From my research, the difference in using All this is based on my interpretation of from these Wikipedia articles:
(let me know if I'm misinterpreting all of this as well) |
It’s interpreting the input as bits already, the formatter is just taking care of the kilo/mega/… suffix. So if the raw field in the document is bytes, then you need to multiply by 8 (you can do this via formula).. The “(1000)” signifies that it’s doing kilobits, not kibibits. |
If you are using formula, you can do the “normalize by unit” as part of the formula using the respective function (I think it’s called normalize_by_unit) |
So really the distinction is I feel like there really should be 4 formatters:
|
Yeah, I see your point. We don't want to add too many options to this list because we plan to rewrite the way formatting is defined anyway (the bits extension was done specifically for this use case). |
@flash1293 I think for this specific use case, Since the formatting is being refactor, the above is something to keep in mind when formatting "data". |
Implement Network traffic in Hosts table via Lens formula. Instead of the single column NW Traffic as shown in the screenshot below, use two separate columns to calculate Inbound(rx) and Outbound(tx) traffic so the user can sort by either field. The column names should be RX (avg.) and TX (avg.).
#141372 added support for calculating bits.
We want to calculate bits per second. The values should be something like 100Mbit/s, 2Gbit/s, etc. The fields used are host.network.ingress.bytes and host.network.egress.bytes. To see how we calculate bits per second with a gauge field in other places see #134471 (comment)
The text was updated successfully, but these errors were encountered: