Skip to content

Commit 1a78c93

Browse files
thomasqueirozbprontclaude
authored
fix(ci): fix Pulsar TLS integration tests with latest image (#25099)
* fix(ci): pin Pulsar integration test image to 4.1.3 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> * Add PULSAR_PREFIX_advertisedAddress=pulsar * fix(ci): revert Pulsar image pin to latest * Revert to version: latest * Fix test.yml format * Delete obsolete comment --------- Co-authored-by: Pavlos Rontidis <pavlos.rontidis@gmail.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b59c9a2 commit 1a78c93

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

tests/integration/pulsar/config/compose.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ services:
88
- 6650:6650
99
- 6651:6651
1010
environment:
11+
# Since apache/pulsar#25238, standalone mode advertises the container's FQDN
12+
# (the container ID) instead of localhost. Set explicitly so TLS hostname
13+
# verification matches the certificate's DNS SAN.
14+
- PULSAR_PREFIX_advertisedAddress=pulsar
1115
- PULSAR_PREFIX_brokerServicePortTls=6651
1216
- PULSAR_PREFIX_tlsKeyFilePath=/etc/pulsar/certs/pulsar.key.pem
1317
- PULSAR_PREFIX_tlsCertificateFilePath=/etc/pulsar/certs/pulsar.cert.pem

tests/integration/pulsar/config/test.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ env:
77
PULSAR_HOST: pulsar
88

99
matrix:
10-
# TODO: re-enable `latest` once TLS tests pass with newer Pulsar images.
11-
# See https://github.com/vectordotdev/vector/issues/25096
12-
version: ["4.1.3"]
10+
version: [latest]
1311

1412
# changes to these files/paths will invoke the integration test in CI
1513
# expressions are evaluated using https://github.com/micromatch/picomatch

0 commit comments

Comments
 (0)