-
Notifications
You must be signed in to change notification settings - Fork 459
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
Sync list of receivers #128
Comments
I'll take this @jpkrohling |
The current deltas:collectorCurrently update to date. May need to compare settings to make sure everything is still following convention. collector-contrib
|
@jpkrohling - What are your thoughts about the "collector-contrib" receivers? Was your plan to rely on |
No, the generic is just an implementation detail, as most receivers can be managed in the same way. Basically, if the receiver has an |
Perhaps it was implicit in the comment above, but for now, all we care about is the |
Perfect! That's exactly what I was thinking. I'll start tackling the deltas 1-by-1 with individual PRs for each. Should we also split this into multiple issues, @jpkrohling? |
This issue here has the list of receivers already, if we reference this issue in all individual PRs, it should be fine. |
@jpkrohling - Could you add me as an assignee, so I can more easily track this work? |
Done |
Thank you! |
@jpkrohling and community - Quick update: I'm planning to start this work either later this week or early next week. My current OSS backlog includes finishing the Loki exporter ( open-telemetry/opentelemetry-collector-contrib#1894 ), adding a small feature to the F5 Cloud exporter, then circling back to this. |
I can take this up! :) |
It's yours, @VineethReddy02! |
This is the list of receivers from contrib repo Below receivers already exist in the operator
Delta receivers that aren't scrapers
Scrapers (which should be ignored as we do not expose any ports for scrapers)
|
@jpkrohling, is that issue done, or should we implement the following receivers?
|
@pavolloffay, how should we proceed with that issue? |
I would propose to close this and add components per request. |
The operator knows about a few receivers, but I'm not sure this list is up to date:
https://github.com/open-telemetry/opentelemetry-operator/tree/master/pkg/collector/parser
This ticket is to sync this list.
Current delta:
The text was updated successfully, but these errors were encountered: