Skip to content

Revert "CAMEL-23225: Propagate OpenTelemetry context across thread boundaries in camel-opentelemetry2"#22254

Merged
apupier merged 1 commit intomainfrom
revert-22190-CAMEL-23225
Mar 25, 2026
Merged

Revert "CAMEL-23225: Propagate OpenTelemetry context across thread boundaries in camel-opentelemetry2"#22254
apupier merged 1 commit intomainfrom
revert-22190-CAMEL-23225

Conversation

@apupier
Copy link
Contributor

@apupier apupier commented Mar 25, 2026

Reverts #22190

based on #22190 (comment)
this previous commit requires a bit more scrutiny before being integrated in the main branch.

I added as reviewer:

  • @squakez which reported that this should surely be reverted and worked on OpenTelemetry2
  • @gnodet who created the PR
  • @johnpoth who reviewed the PR and initially reported the JIRA issue

@apupier apupier requested review from gnodet, johnpoth and squakez March 25, 2026 09:11
@github-actions
Copy link
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 build-all, build-dependents, 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.

Copy link
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.

Thanks for taking care!

@apupier
Copy link
Contributor Author

apupier commented Mar 25, 2026

From my understanding, this was a JIRA https://issues.apache.org/jira/browse/CAMEL-23225 created for opentelemtry (1) and based on new architectural design to improve performance with opentelemetry2, this shouldn't be done this way nowadays

@apupier
Copy link
Contributor Author

apupier commented Mar 25, 2026

/component-test opentelemetry opentelemetry2

@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2026

🧪 CI tested the following changed modules:

  • components/camel-opentelemetry2
  • dsl/camel-kamelet-main
All tested modules (74 modules)
  • Camel :: All Components Sync point
  • Camel :: All Components Sync point [pom]
  • Camel :: Assembly
  • Camel :: Assembly [pom]
  • Camel :: Catalog :: CSimple Maven Plugin (deprecated) [maven-plugin]
  • Camel :: Catalog :: CSimple Maven Plugin (deprecated) SUCCESS [ 1.519 s]
  • Camel :: Catalog :: Camel Catalog
  • Camel :: Catalog :: Camel Catalog [jar]
  • Camel :: Catalog :: Camel Report Maven Plugin
  • Camel :: Catalog :: Camel Report Maven Plugin [maven-plugin]
  • Camel :: Catalog :: Camel Route Parser
  • Camel :: Catalog :: Camel Route Parser [jar]
  • Camel :: Catalog :: Console
  • Camel :: Catalog :: Console [jar]
  • Camel :: Catalog :: Dummy Component
  • Camel :: Catalog :: Dummy Component [jar]
  • Camel :: Catalog :: Lucene (deprecated)
  • Camel :: Catalog :: Lucene (deprecated) [jar]
  • Camel :: Catalog :: Maven
  • Camel :: Catalog :: Maven [jar]
  • Camel :: Catalog :: Suggest
  • Camel :: Catalog :: Suggest [jar]
  • Camel :: Component DSL
  • Camel :: Component DSL [jar]
  • Camel :: Coverage
  • Camel :: Coverage [pom]
  • Camel :: Docs
  • Camel :: Docs [pom]
  • Camel :: Endpoint DSL
  • Camel :: Endpoint DSL [jar]
  • Camel :: Endpoint DSL :: Support
  • Camel :: Endpoint DSL :: Support [jar]
  • Camel :: Integration Tests
  • Camel :: Integration Tests [jar]
  • Camel :: JBang :: Core
  • Camel :: JBang :: Core [jar]
  • Camel :: JBang :: Integration tests
  • Camel :: JBang :: Integration tests [jar]
  • Camel :: JBang :: MCP
  • Camel :: JBang :: MCP [jar]
  • Camel :: JBang :: Main
  • Camel :: JBang :: Main [jar]
  • Camel :: JBang :: Plugin :: Edit
  • Camel :: JBang :: Plugin :: Edit [jar]
  • Camel :: JBang :: Plugin :: Generate
  • Camel :: JBang :: Plugin :: Generate [jar]
  • Camel :: JBang :: Plugin :: Kubernetes
  • Camel :: JBang :: Plugin :: Kubernetes [jar]
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: Route Parser [jar]
  • Camel :: JBang :: Plugin :: Testing
  • Camel :: JBang :: Plugin :: Testing [jar]
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: JBang :: Plugin :: Validate [jar]
  • Camel :: Kamelet Main
  • Camel :: Kamelet Main [jar]
  • Camel :: Launcher
  • Camel :: Launcher [jar]
  • Camel :: Launcher :: Container
  • Camel :: Launcher :: Container [pom]
  • Camel :: Observability Services
  • Camel :: Observability Services [jar]
  • Camel :: Opentelemetry 2
  • Camel :: Opentelemetry 2 [jar]
  • Camel :: YAML DSL
  • Camel :: YAML DSL [jar]
  • Camel :: YAML DSL :: Deserializers
  • Camel :: YAML DSL :: Deserializers [jar]
  • Camel :: YAML DSL :: Maven Plugins
  • Camel :: YAML DSL :: Maven Plugins [maven-plugin]
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator [jar]
  • Camel :: YAML DSL :: Validator Maven Plugin
  • Camel :: YAML DSL :: Validator Maven Plugin [maven-plugin]

@github-actions
Copy link
Contributor

/component-test opentelemetry opentelemetry2 tests passed successfully.

Copy link
Member

@johnpoth johnpoth left a comment

Choose a reason for hiding this comment

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

Let's revert this then - I'll create a seperate JIRA if the problem still occurs in camel-opentelemetry2

@apupier apupier merged commit 3342e30 into main Mar 25, 2026
5 checks passed
@apupier apupier deleted the revert-22190-CAMEL-23225 branch March 25, 2026 10:47
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.

5 participants