Skip to content

Conversation

@adhraj12
Copy link
Contributor

@adhraj12 adhraj12 commented Dec 24, 2025

Description

Resolves #3111.

The automq-metrics module enables the OTLP HTTP exporter but fails to provide a concrete HTTP sender implementation, leading to a crash on startup when OTLP is enabled. The opentelemetry-exporter-otlp artifact does not include a sender by default and expects one to be available on the classpath.

This PR:

  • Adds io.opentelemetry:opentelemetry-exporter-sender-jdk:1.40.0 to project dependencies.
  • Injects this dependency into the automq-metrics module to utilize the native JDK 17 HttpClient.

Testing Strategy

  • Build Verification: Ran ./gradlew :automq-metrics:jar to ensure successful compilation.
  • Dependency Verification: Verified that opentelemetry-exporter-sender-jdk is present in the runtime classpath of automq-metrics.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Signed-off-by: Adhiraj <68840640+adhraj12@users.noreply.github.com>
Signed-off-by: Adhiraj <68840640+adhraj12@users.noreply.github.com>
woshigaopp added a commit that referenced this pull request Dec 31, 2025
This test verifies the fix for issue #3111 where enabling OTLP HTTP
metrics exporter caused broker startup failure due to missing HTTP
sender implementation.

The test includes:
- Broker startup verification with OTLP exporter enabled
- Check for absence of AbstractMethodError in logs
- Verification of successful OTLP initialization
- Basic produce/consume functionality test
- Load test to ensure stability under workload

Related PR: #3124
woshigaopp added a commit that referenced this pull request Dec 31, 2025
Test results show PR #3124 successfully fixes the AbstractMethodError issue.
- No AbstractMethodError found in logs
- Broker starts successfully
- Load test passes
- System remains stable

The OTLP HTTP exporter configuration needs adjustment, but the core issue is resolved.
woshigaopp added a commit that referenced this pull request Dec 31, 2025
Both test cases now pass successfully:
- test_broker_startup_with_otlp_enabled: PASS
- test_otlp_exporter_with_load: PASS

PR #3124 fully resolves Issue #3111.
OTLP exporter works correctly with proper URI format.
Copy link
Contributor

@woshigaopp woshigaopp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@woshigaopp woshigaopp self-requested a review January 4, 2026 14:21
@woshigaopp woshigaopp merged commit 451d0da into AutoMQ:main Jan 4, 2026
7 checks passed
@adhraj12 adhraj12 deleted the fix/otlp-startup-crash branch January 4, 2026 14: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.

[BUG] Kafka broker fails to start when enabling OTLP HTTP metrics exporter

2 participants