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

Add support for converting OpenTelemetry Collector components #6419

Closed
40 tasks done
Tracked by #3659
rfratto opened this issue Feb 21, 2024 · 3 comments · Fixed by #6790
Closed
40 tasks done
Tracked by #3659

Add support for converting OpenTelemetry Collector components #6419

rfratto opened this issue Feb 21, 2024 · 3 comments · Fixed by #6790
Assignees
Labels
flow Related to Grafana Agent Flow frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Comments

@rfratto
Copy link
Member

rfratto commented Feb 21, 2024

#6379 introduced a framework for converting OpenTelemetry Collector components, with initial support for converting otlpreceiver and otlpexporter. There are many more components we need to support conversions for. The following lists track OpenTelemetry Collector components used in both static mode or exclusively used in Flow mode. (Flow mode also uses the components from static mode).

NOTE: Some components may not make sense to convert, but are left in this list for completion. Such components can be ignored and checked off.

Components which do not have a direct Flow component, or are used as an implementation detail in static mode, may be ignored and revisited later during work on converting the tracing subsystem for static mode.

General

Preview Give feedback
  1. frozen-due-to-age
    erikbaranowski
  2. frozen-due-to-age
    erikbaranowski

Components used in static mode

Preview Give feedback
  1. frozen-due-to-age
    tpaschalis
  2. frozen-due-to-age
    tpaschalis
  3. frozen-due-to-age
    rfratto
  4. frozen-due-to-age
    rfratto
  5. frozen-due-to-age
    rfratto
  6. frozen-due-to-age
    rfratto
  7. frozen-due-to-age
    tpaschalis
  8. frozen-due-to-age
    rfratto
  9. frozen-due-to-age
    erikbaranowski
  10. frozen-due-to-age
    tpaschalis
  11. frozen-due-to-age
    erikbaranowski
  12. frozen-due-to-age
  13. frozen-due-to-age
  14. frozen-due-to-age
    erikbaranowski
  15. frozen-due-to-age
    rfratto
  16. frozen-due-to-age
    rfratto
  17. frozen-due-to-age
  18. frozen-due-to-age
  19. frozen-due-to-age
  20. frozen-due-to-age
    erikbaranowski

Components used exclusively in flow mode

Preview Give feedback
  1. frozen-due-to-age
    tpaschalis
  2. frozen-due-to-age
    tpaschalis
  3. frozen-due-to-age
    tpaschalis
  4. frozen-due-to-age
    rfratto
  5. frozen-due-to-age
    tpaschalis
  6. frozen-due-to-age
    tpaschalis
  7. frozen-due-to-age
    tpaschalis
  8. frozen-due-to-age
    tpaschalis
  9. frozen-due-to-age
    tpaschalis
  10. frozen-due-to-age
    tpaschalis
  11. frozen-due-to-age
    tpaschalis
  12. frozen-due-to-age
    tpaschalis
  13. frozen-due-to-age
    tpaschalis
  14. frozen-due-to-age
  15. frozen-due-to-age
    erikbaranowski
  16. frozen-due-to-age
@rfratto
Copy link
Member Author

rfratto commented Feb 21, 2024

Connectors and extensions may need tweaks to the conversion framework before they're implemented; the low hanging fruits right now are converting receivers/processors/exporters.

@tpaschalis
Copy link
Member

Seems like we removed lokiexporter from the list but didn't close the issue, was there any reason behind this or was it unintentional? (I've opened a PR to convert it either way) cc @rfratto

@rfratto
Copy link
Member Author

rfratto commented Feb 23, 2024

Seems like we removed lokiexporter from the list but didn't close the issue, was there any reason behind this or was it unintentional? (I've opened a PR to convert it either way) cc @rfratto

I think this was a bug with the task list; I've only removed duplicate issues from the list, but the eventual consistency of it threw me off and I ended up removing the same item twice, which might've accidentally removed other things. I'll add it back, thanks for the heads up 👍

@erikbaranowski erikbaranowski self-assigned this Feb 27, 2024
@tpaschalis tpaschalis self-assigned this Mar 12, 2024
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label May 12, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
flow Related to Grafana Agent Flow frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants