Skip to content

CAMEL-25781: camel-azure-eventhubs - add 4.22.3 upgrade guide note to main - #28238

Merged
davsclaus merged 1 commit into
apache:mainfrom
gnodet:CAMEL-25781-upgrade-guide-4.22.3
Oct 21, 2026
Merged

davsclaus merged 1 commit into
apache:mainfrom
gnodet:CAMEL-25781-upgrade-guide-4.22.3

Conversation

@gnodet

@gnodet gnodet commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Adds the upgrade guide note for CAMEL-25781 to the 4.22.2 to 4.22.3 section of main's camel-4x-upgrade-guide-4_22.adoc.

As requested by @davsclaus in the review of #28207 (backport PR).

The same note is in the 4.22.x backport PR (#28207) under its own == Upgrading from 4.22.2 to 4.22.3 section.

@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 github-actions Bot added the docs label Oct 21, 2026

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! The heading length is consistent with existing entries in the file (which already include headings up to 109 characters). No issues found.

This review was generated by an AI agent, Hermès on behalf of @gnodet.

@davsclaus davsclaus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Guillaume. The entry is identical to the one in #28207 and to the 4.24 entry, and it sits in the existing Upgrading from 4.22.2 to 4.22.3 section of main's 4_22 guide. LGTM.

A related sync, either in this PR or as a follow-up: main's camel-4x-upgrade-guide-4_18.adoc (4.18.5 to 4.18.6, eventhubs entry, added in 34af3c4) still has the old last paragraph, "Checkpoint failures triggered by batch size are still logged at WARN". #28206 later corrected it in 9c1f25a. The corrected text is right, because on camel-4.18.x the batch-size failure was logged at DEBUG before #28206. Could you bring main's 4_18 copy in line with it?

This review was generated by an AI agent (Claude Code on behalf of davsclaus) and may contain inaccuracies. Please verify all suggestions before applying.

@github-actions

Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • docs

🔬 Scalpel shadow comparison — Scalpel: 0 of 707 tested, 0 compile-only — current: 0 all tested

Maveniverse Scalpel detected 0 affected modules (current approach: 0).

Skip-tests mode would test 0 modules (1 direct + 0 downstream), skip tests for 0 (generated code, meta-modules)

Modules Scalpel would test (0)

ℹ️ Shadow mode — Scalpel observes but does not affect test execution. Learn more


⚙️ View full build and test results

@davsclaus davsclaus added this to the 4.24.0 milestone Oct 21, 2026
@davsclaus
davsclaus merged commit 4b7f8f9 into apache:main Oct 21, 2026
7 checks passed
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