Skip to content

CAMEL-23498: Cleanup remaining SLF4J 1.x artifacts after upgrade to 2.0.x#23186

Merged
davsclaus merged 1 commit into
mainfrom
slf4j
May 13, 2026
Merged

CAMEL-23498: Cleanup remaining SLF4J 1.x artifacts after upgrade to 2.0.x#23186
davsclaus merged 1 commit into
mainfrom
slf4j

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

Summary

  • Replace log4j-slf4j-impl (SLF4J 1.x Log4j2 binding) with log4j-slf4j2-impl (SLF4J 2.x) in 9 component test dependencies and 2 exclusions
  • Fix hardcoded jcl-over-slf4j version 1.7.36 in camel-launcher to use ${slf4j-version}

These are remnants missed during the SLF4J 1.7.x → 2.0.x upgrade (CAMEL-18932).

Modules updated (test dependency log4j-slf4j-impllog4j-slf4j2-impl)

  • camel-zeebe, camel-parquet-avro, camel-flowable, camel-optaplanner, camel-solr, camel-datasonnet, camel-swift, camel-camunda, camel-pulsar

Exclusions updated (stale artifact name)

  • camel-milvus, camel-kudu

Version fix

  • dsl/camel-jbang/camel-launcher/pom.xml: jcl-over-slf4j-version changed from 1.7.36 to ${slf4j-version}

Test plan

  • Full reactor build passes (mvn clean install -DskipTests)
  • CI build passes

Claude Code on behalf of Claus Ibsen

🤖 Generated with Claude Code

….0.x

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
@davsclaus davsclaus requested review from aldettinger and oscerd May 13, 2026 09:20
@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

🧪 CI tested the following changed modules:

  • components/camel-ai/camel-milvus
  • components/camel-camunda
  • components/camel-datasonnet
  • components/camel-flowable
  • components/camel-kudu
  • components/camel-optaplanner
  • components/camel-parquet-avro
  • components/camel-pulsar
  • components/camel-solr
  • components/camel-swift
  • components/camel-zeebe
  • dsl/camel-jbang/camel-launcher

POM dependency changes: targeted tests included

Modules affected by dependency changes (12)
  • :camel-camunda
  • :camel-datasonnet
  • :camel-flowable
  • :camel-kudu
  • :camel-launcher
  • :camel-milvus
  • :camel-optaplanner
  • :camel-parquet-avro
  • :camel-pulsar
  • :camel-solr
  • :camel-swift
  • :camel-zeebe

💡 Manual integration tests recommended:

You modified dsl/camel-jbang/camel-launcher. The related integration tests in dsl/camel-jbang/camel-jbang-it are excluded from CI. Consider running them manually:

mvn verify -f dsl/camel-jbang/camel-jbang-it -Djbang-it-test
All tested modules (21 modules)
  • Camel :: AI :: LangChain4j :: Embedding
  • Camel :: AI :: LangChain4j :: EmbeddingStore
  • Camel :: AI :: Milvus
  • Camel :: Camunda
  • Camel :: DataSonnet
  • Camel :: Flowable
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: TUI
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Kudu
  • Camel :: Launcher
  • Camel :: Launcher :: Container
  • Camel :: OptaPlanner
  • Camel :: Parquet Avro
  • Camel :: Pulsar
  • Camel :: SWIFT
  • Camel :: Solr
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin
  • Camel :: Zeebe (deprecated)

⚙️ View full build and test results

@davsclaus davsclaus merged commit ec76f9f into main May 13, 2026
6 checks passed
@davsclaus davsclaus deleted the slf4j branch May 13, 2026 10:40
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.

3 participants