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

[v23.3.x] Metrics for transform logging #16913

Conversation

oleiman
Copy link
Member

@oleiman oleiman commented Mar 6, 2024

Backport of PR #16566

Closes #16895

@oleiman oleiman added this to the v23.3.x-next milestone Mar 6, 2024
@oleiman oleiman added the kind/backport PRs targeting a stable branch label Mar 6, 2024
@oleiman oleiman self-assigned this Mar 6, 2024
@oleiman oleiman marked this pull request as ready for review March 6, 2024 03:45
@oleiman
Copy link
Member Author

oleiman commented Mar 6, 2024

diff was a minor #include collision

@oleiman oleiman requested a review from rockwotj March 6, 2024 03:50
logger_probe for tracking metrics specific to individual transform
loggers:
  - data_transforms_logger_events_total
    - Total # of log events emitted by some transform.
  - data_transforms_logger_events_dropped_total
    - Total # of some transform's log events that were dropped due
      to buffer capacity constraint.
  - exported to BOTH /metrics and /public_metrics

manager_probe for tracking metrics generic to the logging::manager:
  - data_transforms_log_manager_buffer_usage_ratio
    - Current occupancy of the logging::manager's queues as a fraction
      of total capacity. [0.0..1.0]
  - data_transforms_log_manager_write_errors_total
    - Total number of failures to produce log events to the transform
      logs topic.
  - exported ONLY to /metrics

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
(cherry picked from commit c31ea7f)
One manager_probe per manager instance, initialized on manager::start

One logger_probe per log source, initialized on first manager::enqueue_log

Also moves some init/deinit logic to log_manager::{start,stop} to avoid
duplicate metric registrations

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
(cherry picked from commit 6d59f92)
- Presence of metrics on both endpoints
- Metrics values in contrived scenarios

Signed-off-by: Oren Leiman <oren.leiman@redpanda.com>
(cherry picked from commit b3a6637)
@oleiman oleiman force-pushed the vbotbuildovich/backport-16566-v23.3.x-28 branch from fcc431f to 3d2d4e3 Compare March 6, 2024 05:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants