CAMEL-24335: update Artemis dependency groupId to org.apache.artemis - #25316
Conversation
Since Artemis 2.5.0, artemis-server, artemis-jakarta-server, artemis-jakarta-client and artemis-amqp-protocol relocated from org.apache.activemq to org.apache.artemis, which was triggering Maven relocation warnings during the build. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
|
🧪 CI tested the following changed modules:
🔬 Scalpel shadow comparison — Scalpel: 20 tested, 28 compile-only — current: 19 all testedMaveniverse Scalpel detected 48 affected modules (current approach: 19).
|
gnodet
left a comment
There was a problem hiding this comment.
Clean, well-scoped groupId fix. All four Artemis artifact references (artemis-server, artemis-jakarta-server, artemis-jakarta-client, artemis-amqp-protocol) correctly updated from org.apache.activemq to org.apache.artemis to match the Artemis TLP relocation. Repo-wide grep confirms no remaining instances of the old groupId for these artifacts. The remaining org.apache.activemq references in camel-activemq and camel-activemq6 are genuine ActiveMQ Classic artifacts and are not part of this migration.
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of @gnodet
Description
Since Artemis 2.5.0, the following artifacts relocated from groupId
org.apache.activemqtoorg.apache.artemis(see https://artemis.apache.org/artemis-tlp-groupid-migration):artemis-serverartemis-jakarta-serverartemis-jakarta-clientartemis-amqp-protocolactivemq-artemis-versionwas already bumped to 2.55.0 (via Dependabot), but thegroupIdin thedependency declarations was never updated, so the build emitted Maven relocation warnings:
This updates the
groupIdtoorg.apache.artemisfor these four dependencies in:test-infra/camel-test-infra-artemis/pom.xmltests/camel-itest/pom.xmlcomponents/camel-paho/pom.xmlalready usedorg.apache.artemisforartemis-mqtt-protocol, confirmingthe new groupId is correct.
Verified with
mvnd clean install -Dquickly -Dci.env.name=local: build succeeds and the relocationwarnings are gone.
Target
mainbranch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTestslocally from root folder and I have committed all auto-generated changes.AI-assisted contributions
Co-authored-bytrailers) and the PR description identifies the AI tool used.Claude Code on behalf of ammachado