-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[DOCS-13090] Add OP utilization metrics #33838
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
Changes from 1 commit
28ab498
663ff36
09bf9f2
f169e95
12773e0
792b987
447528b
03db895
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -77,6 +77,16 @@ Data dropped intentionally or unintentionally | |
| : **Description**: The number of events dropped. **Note**: To break down this metric, use the `intentional:true` tag to filter for events that are intentionally dropped or the `intentional:false` tag for events that are not intentionally dropped. | ||
| : **Available for**: Sources, processors, and destinations. | ||
|
|
||
| Timed out events | ||
| : **Metric**: `pipelines.component_timed_out_events_total` | ||
| : **Description**: The number of events that timed out. | ||
|
maycmlee marked this conversation as resolved.
Outdated
|
||
| : **Available for**: Sources, processors, and destinations. | ||
|
maycmlee marked this conversation as resolved.
Outdated
|
||
|
|
||
| Time out requests | ||
|
maycmlee marked this conversation as resolved.
Outdated
|
||
| : **Metric**: `pipelines.component_timed_out_requests_total` | ||
| : **Description**: The number of requests that timed out. | ||
|
maycmlee marked this conversation as resolved.
Outdated
|
||
| : **Available for**: Sources, processors, and destinations. | ||
|
maycmlee marked this conversation as resolved.
Outdated
|
||
|
|
||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This can be addressed in a different PR, but visually, I wonder if it might be best to emulate the Buffer metrics (when buffering is enabled) format without mentioning
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hi @iadjivon! I don't quite understand. Are you saying to put the metric name as the header instead of |
||
| Utilization | ||
| : **Metric**: `pipelines.utilization` | ||
| : **Description**: The component's activity. A value of `0` indicates an idle component that is waiting for input. A value of `1` indicates a component that is never idle, which means that the component is likely a bottleneck in the processing topology that is creating backpressure, which might cause events to be dropped. | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -19,4 +19,22 @@ Track buffer behavior with these additional metrics: | |||||
| : Total bytes successfully flushed from the buffer. | ||||||
|
|
||||||
| `pipelines.buffer_discarded_events_total` | ||||||
| : Events discarded from the buffer (for example, due to overflow). | ||||||
| : Events discarded from the buffer (for example, due to overflow). | ||||||
|
maycmlee marked this conversation as resolved.
Outdated
|
||||||
|
|
||||||
| `pipelines.source_buffer_max_event_size` | ||||||
| : Maximum event size for a source's buffer. | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is an unfortunate choice of metric name, as it actually means the maximum buffer size in terms of number of events. It probably should have been
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note that this followed the Vector buffer metric naming spec which contradicts the Vector metric naming spec
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ahh okay, that actually makes more sense. If you and Vector want to make the name change, let me know if you I should hold off publishing this.
Suggested change
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removed these metrics since the names are getting changed and released with 2.13 release:
|
||||||
|
|
||||||
| `pipelines.source_buffer_utilization` | ||||||
| : The percentage of a source's buffer that is used. | ||||||
|
|
||||||
| `pipelines.source_buffer_utilization_level` | ||||||
| : Utilization level for a source's buffer. | ||||||
|
maycmlee marked this conversation as resolved.
Outdated
|
||||||
|
|
||||||
| `pipelines.transform_buffer_max_event_size` | ||||||
| : Max event size for a transform's buffer. | ||||||
|
|
||||||
| `pipelines.transform_buffer_utilization` | ||||||
| : The percentage of a transform's buffer that is used. | ||||||
|
|
||||||
| `pipelines.transform_buffer_utilization_level` | ||||||
| : Utilization level for a transform's buffer. | ||||||
|
maycmlee marked this conversation as resolved.
Outdated
|
||||||
Uh oh!
There was an error while loading. Please reload this page.