Skip to content

chore: upgrade Pulsar client to 4.2.0 and fix V1 topic names in tests#22553

Open
Croway wants to merge 1 commit intoapache:mainfrom
Croway:quick-fix/pulsar-4.2.0-upgrade
Open

chore: upgrade Pulsar client to 4.2.0 and fix V1 topic names in tests#22553
Croway wants to merge 1 commit intoapache:mainfrom
Croway:quick-fix/pulsar-4.2.0-upgrade

Conversation

@Croway
Copy link
Copy Markdown
Contributor

@Croway Croway commented Apr 12, 2026

Pulsar 4.2.0 (PIP-457) removed V1 topic name support. Fix test topic URIs that used slashes in the topic name portion, and document the change in the migration guide.

Pulsar 4.2.0 standalone no longer defaults advertisedAddress to localhost, causing the broker to advertise the container hostname. Set PULSAR_PREFIX_advertisedAddress=localhost in the test container.

Pulsar 4.2.0 (PIP-457) removed V1 topic name support. Fix test topic
URIs that used slashes in the topic name portion, and document the
change in the migration guide.

Pulsar 4.2.0 standalone no longer defaults advertisedAddress to
localhost, causing the broker to advertise the container hostname.
Set PULSAR_PREFIX_advertisedAddress=localhost in the test container.
@Croway Croway requested a review from oscerd April 12, 2026 17:16
@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.

* The `local-kafka3-container` mapping has been renamed to `local-kafka-container`. If you were using `-Dkafka.instance.type=local-kafka3-container`, update to `-Dkafka.instance.type=local-kafka-container`.
* The Strimzi test service now runs in KRaft mode (no ZooKeeper). The `ZookeeperContainer` class has been removed.

=== camel-pulsar
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@davsclaus is this section ok here, or should it be in the 4_20 upgrade guide?

@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • components/camel-pulsar
  • docs
  • parent
  • test-infra/camel-test-infra-pulsar

POM dependency changes: targeted tests included

Changed properties: pulsar-version

Modules affected by dependency changes (3)
  • :camel-pulsar
  • :camel-test-infra-pulsar
  • :docs
All tested modules (10 modules)
  • Camel :: Docs
  • Camel :: JBang :: MCP
  • Camel :: JBang :: Plugin :: Route Parser
  • Camel :: JBang :: Plugin :: Validate
  • Camel :: Launcher :: Container
  • Camel :: Pulsar
  • Camel :: Test Infra :: All test services
  • Camel :: Test Infra :: Pulsar
  • Camel :: YAML DSL :: Validator
  • Camel :: YAML DSL :: Validator Maven Plugin

⚙️ View full build and test results

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.

2 participants