-
Notifications
You must be signed in to change notification settings - Fork 183
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
OpenTelemetry Agent support #1027
Conversation
@pmm-sumo What is the purpose of the opentelemetry agent? |
Essentially, this makes scaling much more efficient. Agent gets the local connections, buffers data and sends those to collector. Eventually, tagging should move on there as well: https://opentelemetry.io/docs/collector/about/#agent |
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.
One nit, otherwise I'm 👍
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.
Remove auto-generated files due to #1052
When `otelagent.enabled=true`, deploys a DeamonSet with OpenTelemetry Collector running in Agent mode, configured to send data to the OpenTelemetry Collector instance.
Co-authored-by: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com>
Sorry for the delay. The requests came when I was on a sick-leave and missed them. In either case - prepared the fixes. Going to test them bit later. But so far - does it look OK now @sumo-drosiek ? |
@pmm-sumo yup, this is fine. Just one another ask, please update README.md with the new configuration 😅 |
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.
Other than those minor nitpicks I believe this is 👍🏻
Co-authored-by: Patryk Małek <69143962+pmalek-sumo@users.noreply.github.com>
Co-authored-by: Patryk Małek <69143962+pmalek-sumo@users.noreply.github.com>
Description
When
otelagent.enabled=true
, deploys a DeamonSet with OpenTelemetry Collector running in Agent mode, configured to send data to the OpenTelemetry Collector instance.Testing performed