Repository navigation
Upgrade to Camel 4.18.4 - #742
Merged
Merged
Conversation
Author
|
Tested this branch against an application of mine (local build):
This application is quite simple: it exposes a REST API that receives a JSON payload and spreads it across multiple queues, which then forward the message to several endpoints (typical ESB). I ran the resulting distribution in a local environment: all features install cleanly, and I verified end-to-end that messages flow through ActiveMQ and my Camel routes as expected. |
jbonofre
self-requested a review
August 27, 2026 06:44
Member
|
Thanks! I would like to merge the other fixes before the update (in order also to test on CI). |
This was referenced Sep 3, 2026
Closed
jbonofre
added a commit
that referenced
this pull request
Sep 6, 2026
…still serves (#750) Backport of #746. CamelGooglePubsubITest on this branch still pins gcr.io/google.com/cloudsdktool/google-cloud-cli:441.0.0-emulators. That tag still resolved on 2026-08-28 -- the CI run for PR #742 pulled it successfully -- but gcr.io keeps only a rolling window of the most recent emulator tags, so the pin has since stopped resolving. The failure mode is the one #746 described on main: Testcontainers fails the container fetch with "manifest unknown", the itest errors out in initializationError, and the reactor halts at camel-google-pubsub-test, skipping every later module. That blocks meaningful CI on the 4.18.x line, including PR #742. Bump it to 583.0.0-emulators and keep the comment recording the registry retention behaviour, so the next bump is not a mystery. The other four pinned test images (cp-kafka, influxdb, localstack, rabbitmq) are untouched.
Member
|
Let me rebase as I fixed the issue with itest on google pubsub. |
Author
yep thanks |
jbonofre
force-pushed
the
camel-karaf-4.18.x
branch
from
September 7, 2026 09:21
992f454 to
6d948ab
Compare
…73 logging
Camel 4.18.4 upgrades google-cloud-pubsub, which pulls gax 2.73.x. Its
com.google.api.gax.logging.LogData has a static (class-init) dependency on
com.google.gson.Gson, and the gRPC/HTTP logging interceptors touch LogData
on the first call. The camel-google-pubsub, camel-google-pubsub-lite and
camel-google-secret-manager features wrap com.google.api/gax without an
optional-import override and do not provide a gson bundle, so at runtime
every call fails with:
NoClassDefFoundError: Could not initialize class com.google.api.gax.logging.LogData
Caused by: ClassNotFoundException: com.google.gson.Gson not found by wrap_com.google.api.gax
CamelGooglePubsubITest.testResultMock then times out with 0 messages on
the mock. Add mvn:com.google.code.gson/gson to those three features
(gson ${gson-version} is already used by ~10 other features).
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011mhjfmJXTtsV58PBdvxvSX
Member
|
@lobtx I fixed everything on your PR. Let me approve and merge, I will move forward with the release then. |
jbonofre
approved these changes
Sep 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Used the tooling to list and apply changes from camel 4.18.4.
I made some fixes due to this upgrade:
camel-google-secret-manager: applied https://issues.apache.org/jira/browse/CAMEL-23729 => protobuf rolled-back to ${protobuf-version} instead of ${protobuf4-version} due to google-cloud-secretmanager downgrade to 2.82.0 in Camel 4.18.3tests: renamed camel headersfeatures: addedcom.google.code.gson/gsontocamel-google-pubsub,camel-google-pubsub-liteandcamel-google-secret-manager. Camel 4.18.4 pulls gax 2.73.x, whosecom.google.api.gax.logging.LogDatahas a static dependency on Gson (loaded by the gRPC/HTTP logging interceptors). Without it every gax call fails withNoClassDefFoundError/ClassNotFoundException: com.google.gson.Gson not found by wrap_com.google.api.gax, which madeCamelGooglePubsubITesttime out with 0 messages on CI. These three features wrapcom.google.api/gaxplainly (no optional-import override) and did not already provide gson. Diagnosed and committed by Claude Code on behalf of JB Onofré.