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

Why is intervalprocessor not part of contrib Distribution (or any other) and what is the alternative? #36514

Open
genadipost opened this issue Nov 24, 2024 · 8 comments · May be fixed by #36886
Assignees

Comments

@genadipost
Copy link

Component(s)

processor/interval

Describe the issue you're reporting

Why is intervalprocessor not part of contrib Distribution (or any other).
What would be the alternative if i would like to aggregate all metrics (that can be aggregated) over certain period of time.

@genadipost genadipost added the needs triage New item requiring triage label Nov 24, 2024
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@VihasMakwana
Copy link
Contributor

@genadipost which version are you using?
From what I can see, the processor/intervalprocessor is a part of builder-config.yaml
Could you explain the question in more detail?

@VihasMakwana VihasMakwana removed the needs triage New item requiring triage label Nov 25, 2024
@ChrsMark
Copy link
Member

I think that's a valid issue with contrib distro:

error decoding 'processors': unknown type: "intervalprocessor" for id: "intervalprocessor" (valid values: [experimental_metricsgeneration metricstransform probabilistic_sampler resource batch attributes filter groupbytrace routing logdedup remotetap sumologic tail_sampling groupbyattrs k8sattributes span memory_limiter cumulativetodelta deltatocumulative deltatorate geoip redaction resourcedetection transform])

I see it's not part of https://github.com/open-telemetry/opentelemetry-collector-releases/blob/main/distributions/otelcol-contrib/manifest.yaml indeed.

Since it's stability is still development I guess code-owners would need to evaluate if this component can make it to beta (not sure if there are specific requirements for components to be included in contrib distro). @RichieSams @sh0rez @djaglowski any plans/thoughts on this?

The alternative would be to build a custom Collector following the steps from https://opentelemetry.io/docs/collector/custom-collector/.

@VihasMakwana
Copy link
Contributor

Thanks for pointing it out @ChrsMark. I completely missed that aspect.

@genadipost
Copy link
Author

genadipost commented Nov 26, 2024

@ChrsMark @VihasMakwana I would like aggregate all metrics over 5 minutes. Could i get the same functionality by using Metrics Transform Processor or other processor?

@RichieSams
Copy link
Contributor

I think it's ok to include in beta

@genadipost
Copy link
Author

@ChrsMark @VihasMakwana @RichieSams Should I open a ticket?

@RichieSams
Copy link
Contributor

No, we can use this issue as a tracker. This issue can be assigned to me.
I've been behind trying to finish all my work issues before the end of the quarter. But I can do this change this week

RichieSams added a commit to RichieSams/opentelemetry-collector-contrib that referenced this issue Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants