Skip to content

Remove batch processor references from OpenTelemetry Collector configurations#803

Merged
trask merged 2 commits into
mainfrom
copilot/fix-802
Sep 10, 2025
Merged

Remove batch processor references from OpenTelemetry Collector configurations#803
trask merged 2 commits into
mainfrom
copilot/fix-802

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Sep 10, 2025

This PR removes references to the batch processor from OpenTelemetry Collector configurations and moves the batch functionality to the exporters' sending_queue configuration, following the updated OpenTelemetry Collector architecture.

Changes Made

Files Modified

  • otlp/docker/otel-collector-config-demo.yaml
  • logging-k8s-stdout-otlp-json/k8s/collector-configmap.yaml

Transformation Applied

  • Removed batch: from the processors: section
  • Removed batch from processor lists in service pipelines
  • Added sending_queue: with batch: configuration to relevant exporters:
    • prometheus exporter
    • otlp/jaeger exporter
    • otlphttp/metrics exporter
    • otlphttp/traces exporter
    • otlphttp/logs exporter

Files Unchanged (False Positives)

The following files contain batch references but are SDK declarative configurations (not Collector configurations) and were intentionally left unchanged:

  • javaagent/sdk-config.yaml
  • declarative-configuration/otel-sdk-config.yaml

Before/After Example

Before:

processors:
  batch:

service:
  pipelines:
    metrics:
      receivers: [otlp]
      processors: [batch]
      exporters: [prometheus]

After:

exporters:
  prometheus:
    endpoint: "0.0.0.0:8889"
    sending_queue:
      batch:

service:
  pipelines:
    metrics:
      receivers: [otlp]
      exporters: [prometheus]

All modified YAML files have been validated for correct syntax.

Fixes #802.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • scans-in.gradle.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:MaxMetaspaceSize=512m --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED -Dfile.encoding=UTF-8 -Duser.country -Duser.language=en -Duser.variant -cp /home/REDACTED/.gradle/wrapper/dists/gradle-9.0.0-bin/d6wjpkvcgsg3oed0qlfss3wgl/gradle-9.0.0/lib/gradle-daemon-main-9.0.0.jar -javaagent:/home/REDACTED/.gradle/wrapper/dists/gradle-9.0.0-bin/d6wjpkvcgsg3oed0qlfss3wgl/gradle-9.0.0/lib/agents/gradle-instrumentation-agent-9.0.0.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 9.0.0 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Sep 10, 2025

@trask 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: trask <218610+trask@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove references to Collector's batch processor in opentelemetry-java-examples Remove batch processor references from OpenTelemetry Collector configurations Sep 10, 2025
Copilot AI requested a review from trask September 10, 2025 15:38
@trask trask marked this pull request as ready for review September 10, 2025 15:53
@trask trask requested review from a team as code owners September 10, 2025 15:53
@trask trask merged commit 50de7b1 into main Sep 10, 2025
19 of 25 checks passed
@trask trask deleted the copilot/fix-802 branch September 10, 2025 16:32
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.

Remove references to Collector's batch processor in opentelemetry-java-examples

3 participants