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

[processor/interval] unavailable in 0.118.0 #37501

Closed
wojciechkaleb opened this issue Jan 27, 2025 · 5 comments
Closed

[processor/interval] unavailable in 0.118.0 #37501

wojciechkaleb opened this issue Jan 27, 2025 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed processor/interval

Comments

@wojciechkaleb
Copy link

wojciechkaleb commented Jan 27, 2025

Component(s)

[processor/interval]

What happened?

Description

I'm getting bug that processors/invervalprocessor is unknown type in the processor context

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

I'm using opentelemetry-collector-contrib 0.118.0 and as mentioned in the release notes there should be as the new component.
https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.118.0

Is there anything more than the collector configuration I should adjust?

processors:
  intervalprocessor:

Collector version

0.118.0

Environment information

Environment

OS: (e.g., "Ubuntu 20.04")

OpenTelemetry Collector configuration

extensions:
  basicauth/traces:
    client_auth:
      username: '${TEMPO_USER}'
      password: '${TEMPO_PASSWORD}'
  basicauth/metrics:
    client_auth:
      username: '${PROM_USER}'
      password: '${PROM_PASSWORD}'
  basicauth/logs:
    client_auth:
      username: '${LOKI_USER}'
      password: '${LOKI_PASSWORD}'
  health_check:
    endpoint: 0.0.0.0:${HEALTHCHECK_PORT}
receivers:
  otlp:
    protocols:
      http:
        endpoint: 0.0.0.0:${HTTP_PORT}

processors:
  batch/metrics:
    timeout: 10s
    send_batch_max_size: 20000
  batch/traces:
  intervalprocessor:


exporters:
  otlp:
    endpoint: '${TEMPO_ENDPOINT}'
    auth:
      authenticator: basicauth/traces
  prometheusremotewrite:
    endpoint: '${PROM_ENDPOINT}'
    auth:
      authenticator: basicauth/metrics

  loki:
    endpoint: '${LOKI_ENDPOINT}'
    auth:
      authenticator: basicauth/logs

service:
  extensions:
    [health_check, basicauth/traces, basicauth/metrics, basicauth/logs]
  pipelines:
    traces:
      receivers: [otlp]
      processors: [batch/traces, tail_sampling]
      exporters: [otlp]
    metrics:
      receivers: [otlp]
      processors:
        [
          intervalprocessor,
          batch/metrics,
        ]
      exporters: [prometheusremotewrite]
    logs:
      receivers: [otlp]
      exporters: [loki]

Log output

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

Additional context

Not sure if I understand your distribution process correctly and all the dependencies between projects but I've checked the manifest.yml of 0.118.0 and there is no processors/intervalprocessor mentioned
https://github.com/open-telemetry/opentelemetry-collector-releases/blob/main/distributions/otelcol-contrib/manifest.yaml

@wojciechkaleb wojciechkaleb added bug Something isn't working needs triage New item requiring triage labels Jan 27, 2025
@wojciechkaleb wojciechkaleb changed the title processors/intervalprocessor unavailable in 0.118.0 [processors/intervalprocessor] unavailable in 0.118.0 Jan 27, 2025
@github-actions github-actions bot added the help wanted Extra attention is needed label Jan 27, 2025
@wojciechkaleb wojciechkaleb changed the title [processors/intervalprocessor] unavailable in 0.118.0 [processor/intervalprocessor] unavailable in 0.118.0 Jan 27, 2025
@wojciechkaleb wojciechkaleb changed the title [processor/intervalprocessor] unavailable in 0.118.0 [processor/interval]unavailable in 0.118.0 Jan 27, 2025
@wojciechkaleb wojciechkaleb changed the title [processor/interval]unavailable in 0.118.0 [processor/interval] unavailable in 0.118.0 Jan 27, 2025
@wojciechkaleb
Copy link
Author

Some problems with adding labels, sorry for pinging directly :|
@RichieSams @sh0rez

Copy link
Contributor

Pinging code owners for processor/interval: @RichieSams @sh0rez. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label.

@crobert-1
Copy link
Member

crobert-1 commented Jan 27, 2025

The stability is noted as alpha, so this component should be available in the contrib releases. However, it's not included in the contrib manifest, which means it's not being included. It needs to be added there.

Removing needs triage as this is a valid issue.

@RichieSams
Copy link
Contributor

PR created here: open-telemetry/opentelemetry-collector-releases#804

github-merge-queue bot pushed a commit to open-telemetry/opentelemetry-collector-releases that referenced this issue Jan 28, 2025
* Update contrib and k8s distributions to match main repo

Solves open-telemetry/opentelemetry-collector-contrib#37501

* Add changelog

* Update .chloggen/add_interval_processor.yaml

Co-authored-by: Curtis Robert <crobert@splunk.com>

* Update .chloggen/add_interval_processor.yaml

---------

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Co-authored-by: Curtis Robert <crobert@splunk.com>
@crobert-1
Copy link
Member

Since open-telemetry/opentelemetry-collector-releases#804 has been merged, I'm marking this completed. The interval processor will be included in release v0.119.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed processor/interval
Projects
None yet
Development

No branches or pull requests

3 participants