Skip to content

fix(ci): fix Pulsar TLS integration tests with latest image#25099

Merged
thomasqueirozb merged 7 commits intomasterfrom
fix/pulsar-tls-advertised-address
Apr 2, 2026
Merged

fix(ci): fix Pulsar TLS integration tests with latest image#25099
thomasqueirozb merged 7 commits intomasterfrom
fix/pulsar-tls-advertised-address

Conversation

@thomasqueirozb
Copy link
Copy Markdown
Contributor

Summary

Since apache/pulsar#25238, Pulsar standalone mode advertises the container's FQDN (the container ID) instead of localhost. The Pulsar client reconnects to the advertised address during service discovery, causing TLS hostname verification to fail against the certificate's DNS:pulsar SAN.

Setting advertisedAddress=pulsar explicitly fixes the mismatch and allows reverting the image pin back to latest.

Vector configuration

NA

How did you test this PR?

Ran ./scripts/run-integration-test.sh -r 0 int pulsar locally -- all 7 tests pass (including the two TLS tests that were failing).

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

References

pront and others added 3 commits April 1, 2026 15:19
The `apachepulsar/pulsar:latest` image moved to a version that breaks
TLS connectivity, causing `pulsar_happy_tls` and
`consumes_event_with_tls` to fail consistently in CI.

Pin to 4.1.3 to unblock the merge queue. Re-enabling `latest` is
tracked in #25096.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Apr 1, 2026
@thomasqueirozb thomasqueirozb changed the base branch from pront-disable-pulsar-integration-tests to master April 1, 2026 20:08
@thomasqueirozb thomasqueirozb marked this pull request as ready for review April 1, 2026 20:26
@thomasqueirozb thomasqueirozb requested a review from a team as a code owner April 1, 2026 20:26
@thomasqueirozb thomasqueirozb enabled auto-merge April 1, 2026 20:49
@thomasqueirozb thomasqueirozb disabled auto-merge April 2, 2026 17:39
@thomasqueirozb thomasqueirozb enabled auto-merge April 2, 2026 17:39
@thomasqueirozb thomasqueirozb added this pull request to the merge queue Apr 2, 2026
Merged via the queue into master with commit 1a78c93 Apr 2, 2026
59 checks passed
@thomasqueirozb thomasqueirozb deleted the fix/pulsar-tls-advertised-address branch April 2, 2026 18:39
@github-actions github-actions bot locked and limited conversation to collaborators Apr 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pulsar TLS integration tests consistently failing in CI

2 participants