Fix #2830: Remove deprecated Camel endpoint options from pulsar-sink and slack-sink#2886
Merged
Conversation
…-sink and slack-sink Audited the catalog against the Camel 4.21.0 component catalog. Two Kamelets set endpoint options that Camel marks deprecated: - pulsar-sink: maxPendingMessagesAcrossPartitions (deprecated on camel-pulsar). Removed the optional property and its template mapping; the non-deprecated maxPendingMessages option is already exposed and remains. - slack-sink: iconEmoji and iconUrl (deprecated on camel-slack, since Slack no longer supports API-side bot icon overrides for apps). Removed both optional properties and their mappings. The remaining deprecated *component* usages found by the audit (json-patch, paho/MQTT v3, splunk) have no safe drop-in replacement and are left for a PMC decision; see the PR description. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
davsclaus
approved these changes
Jun 26, 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.
Summary
Addresses #2830 — audit the catalog for deprecated Camel component/option usage on the move to 4.21.0.
Audited the catalog against the Camel 4.21.0-SNAPSHOT component catalog (
camel catalog component --jsonplus thecamel-catalogoption metadata), covering all 109 used components and the options the Kamelets actually set. Two Kamelets set deprecated endpoint options — fixed here. The deprecated components found by the audit have no safe drop-in replacement and are listed below for a PMC decision rather than a breaking change.Fixed in this PR
pulsar-sinkmaxPendingMessagesAcrossPartitionscamel-pulsar. The non-deprecatedmaxPendingMessagesis already exposed in the same Kamelet and remains.slack-sinkiconEmoji,iconUrlcamel-slack(Slack removed API-side bot icon overrides for modern apps). No replacement option.Both were optional properties; removed the property declarations and their template mappings only. No functional change to the core behaviour of either Kamelet.
Audit findings that need a PMC decision (NOT changed here)
These Kamelets use deprecated components with no safe drop-in replacement — flagging rather than making a breaking/behavioural change in this PR:
json-patch-action→camel:json-patch(deprecated): no replacement component exists in the 4.21 catalog.mqtt-source,mqtt-sink→camel:paho(MQTT v3, deprecated): the only sibling iscamel:paho-mqtt5(MQTT v5 — a protocol change), andmqtt5-source/mqtt5-sinkalready cover v5. Migrating would break v3 users / duplicate the v5 Kamelets.splunk-sink,splunk-source→camel:splunk(deprecated):splunk-hecis publish-only and config-incompatible;splunk-source(search) has no HEC equivalent, andsplunk-hec-sinkalready covers HEC publishing.Suggest tracking these in a follow-up issue for a PMC call (deprecate the affected Kamelets, accept the behavioural change, or keep them until the components are removed upstream).
Verification
script/validatorover all 250 Kamelets: no errors.mvn clean install(full reactor, with tests): passes.kamelets/is committed; thelibrary/camel-kameletsmirror is regenerated by the post-merge regen bot.AI-generated by Claude Code on behalf of Andrea Cosentino (@oscerd).