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

[Receivers] Add bytes ingressed metrics for receivers #7308

Open
splunkericl opened this issue Mar 3, 2023 · 0 comments
Open

[Receivers] Add bytes ingressed metrics for receivers #7308

splunkericl opened this issue Mar 3, 2023 · 0 comments

Comments

@splunkericl
Copy link
Contributor

splunkericl commented Mar 3, 2023

Is your feature request related to a problem? Please describe.
Currently, the obsreport for receivers only support number of events sent successfully/failed. One new observability our application is looking for is the raw bytes the receiver receives for the events.

This raw bytes should be however much the receiver receives over the network. e.g: compressed bytes sent over HTTP.

Describe the solution you'd like
Updating obsreport to include new fields to include bytes.

// move to a struct so it is easier to extend without breaking signature
type observabilityData struct {
   numReceived int
   err error
   // optional. if nil, do not emit
   byteCount *int64
}

EndTracesOp(receiverCtx context.Context, format string, data observabilityData)
EndLogsOp(receiverCtx context.Context, format string, data observabilityData)
EndMetricsOp(receiverCtx context.Context, format string, data observabilityData)

Describe alternatives you've considered

  • Considered adding this in our processor to count the bytes for each log record. However, this doesn't reflect true bytes received.
  • Another consideration is creating a new extension to provide this functionality. Each receiver can choose to include this extension. However, the extension would act just like obsreport.

Additional Context
receivers equivalent github issue: #7310

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

No branches or pull requests

1 participant