Skip to content

Using OpenTelemetry instrumentation via Spring Boot starter and OpenTelemetry2#22675

Open
beskow wants to merge 4 commits intoapache:mainfrom
beskow:CAMEL-23343-opentelemetry-starter
Open

Using OpenTelemetry instrumentation via Spring Boot starter and OpenTelemetry2#22675
beskow wants to merge 4 commits intoapache:mainfrom
beskow:CAMEL-23343-opentelemetry-starter

Conversation

@beskow
Copy link
Copy Markdown
Contributor

@beskow beskow commented Apr 17, 2026

Description

Document how to configure and use opentelemetry-spring-boot-starter as an alternative to the OpenTelemetry agent (see https://issues.apache.org/jira/browse/CAMEL-23343).

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.

  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

Comment thread components/camel-opentelemetry2/src/main/docs/opentelemetry2.adoc Outdated
…s an alternative to the OpenTelemetry agent.
@beskow beskow force-pushed the CAMEL-23343-opentelemetry-starter branch from 188f98a to f9e3531 Compare April 19, 2026 06:25
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 19, 2026

🧪 CI tested the following changed modules:

  • components/camel-opentelemetry2
All tested modules (9 modules)
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Launcher :: Container
  • Camel :: Observability Services
  • Camel :: Opentelemetry 2
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin

⚙️ View full build and test results

@oscerd oscerd requested a review from squakez April 19, 2026 10:41
Copy link
Copy Markdown
Contributor

@squakez squakez left a comment

Choose a reason for hiding this comment

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

LGTM, just a minor note would be required for compatibility reasons.

<dependency>
<groupId>io.opentelemetry.instrumentation</groupId>
<artifactId>opentelemetry-instrumentation-bom</artifactId>
<version>2.26.1</version>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Probably you want to set here a version compatible with the Opentelemetry dependencies that are used within a given Camel version used. We should add a note.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good point, I'll have added that!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, but I think that note is not enough. I'm adding a comment inline to show how I think it should be. Feel free to accept the inline change if you want.

Copy link
Copy Markdown
Contributor Author

@beskow beskow Apr 21, 2026

Choose a reason for hiding this comment

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

Sure! I had added the same info in the paragraph above, but it might be more clear the way you suggested. Now applied, thanks!

@apupier apupier requested a review from squakez April 21, 2026 06:48
<dependency>
<groupId>io.opentelemetry.instrumentation</groupId>
<artifactId>opentelemetry-instrumentation-bom</artifactId>
<version>2.26.1</version>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, but I think that note is not enough. I'm adding a comment inline to show how I think it should be. Feel free to accept the inline change if you want.

Comment thread components/camel-opentelemetry2/src/main/docs/opentelemetry2.adoc Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants