Skip to content

Update receiverhelper for requests that failed to be received #12802

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

Draft
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

gizas
Copy link

@gizas gizas commented Apr 4, 2025

Description

This is an ongoing work to add :

  • In the endOp function of ObsReport, errors are classified as either:
    • Downstream errors (refused) - when internal.IsDownstreamError(err) returns true
    • Internal errors - when the error is not a downstream error
  • New metrics metricsotelcol_receiver_internal_errors_spans, otelcol_receiver_internal_errors_metric_points and otelcol_receiver_internal_errors_log_records in case of inetrnal errors returned above
  • Support in OTLP receiver to handle IsClientDisconnectError. Network disconnect errors will be reported in otelcol_receiver_internal_errors_metric_points
  • Support of otelcol_receiver_requests metric that will determines the outcome of the request:
    • "success" if there's no error
    • "refused" if there's a downstream error
    • "failure" if there's an internal error

Link to tracking issue

Fixes # #12207

Testing

Only added the integration tests. That is why opened this is as a draft

Documentation

See updated receiver/receiverhelper/documentation.md

gizas added 17 commits April 3, 2025 18:30
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Signed-off-by: Andreas Gkizas <[email protected]>
Copy link

linux-foundation-easycla bot commented Apr 4, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant