-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
enhancement(observability, sinks): Emit events for sent events in all HttpSink sinks #9275
Conversation
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
✔️ Deploy Preview for vector-project ready! 🔨 Explore the source changes: dbefe49 🔍 Inspect the deploy log: https://app.netlify.com/sites/vector-project/deploys/614a6315a3dd280007ee47ea 😎 Browse the preview: https://deploy-preview-9275--vector-project.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Common makes sense to me at least
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of questions, otherwise looks good!
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
…lity Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not crazy about documenting it as being reported by all sinks yet, but hopefully all sinks do report it soon.
Signed-off-by: Bruce Guenter <bruce.guenter@datadoghq.com>
This ended up being pretty simple, as the events flow through only two paths for all these sinks.
After going in to add the metric documentation into each sink, I decided to move it into the common sink documentation, since it will eventually be added to all sinks, and the
events_out_total
is already documented there. If this is not appropriate, I can move it back to each individual sink.Closes #9225