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 interval processor, spanlogs connector, and spanmetrics connector #1119

Merged
merged 8 commits into from
Jan 24, 2025

Conversation

petewall
Copy link
Collaborator

@petewall petewall commented Jan 15, 2025

This change adds the ability to use the following components to the AppO11y pipeline:

  • interval processor
  • spanlogs connector
  • spanmetrics connector

It also moves the memory limiter to just after the receivers.

Finally, it also refactors how the pipeline is built, using a pipeline.tpl file. (this resulted in many components needing to change their .metricsOutput reference to .metrics, etc...)

Added unit tests to show how the pipeline changes when injecting the new components.

@petewall petewall linked an issue Jan 15, 2025 that may be closed by this pull request
@petewall petewall force-pushed the feat/interval-processor branch 2 times, most recently from 36a0a21 to 72a68cf Compare January 23, 2025 19:50
@petewall petewall marked this pull request as ready for review January 23, 2025 21:05
@petewall petewall requested a review from rlankfo January 23, 2025 21:05
@petewall petewall changed the title Add interval processor Add interval processor, spanlogs connector, and spanmetrics connector Jan 23, 2025
@petewall petewall force-pushed the feat/interval-processor branch from 1d3b882 to 76ae190 Compare January 24, 2025 23:43
…ight its potential cause for billing.

Signed-off-by: Pete Wall <[email protected]>
@petewall petewall merged commit 538fe77 into main Jan 24, 2025
33 checks passed
@petewall petewall deleted the feat/interval-processor branch January 24, 2025 23:57
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

Successfully merging this pull request may close these issues.

v2 - please add the interval processor as an option
3 participants