Skip to content

Conversation

@cristianrgreco
Copy link
Collaborator

@cristianrgreco cristianrgreco commented Jun 9, 2025

This fixes a flaky test, where the Kafka client complains that the RF is zero, because there are no brokers yet listening.

@netlify
Copy link

netlify bot commented Jun 9, 2025

Deploy Preview for testcontainers-node ready!

Name Link
🔨 Latest commit 917fea9
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-node/deploys/6846eb1551c6ef0008620277
😎 Deploy Preview https://deploy-preview-1039--testcontainers-node.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@cristianrgreco cristianrgreco changed the title fix flaky kafka test Wait for ZK to be listening when using provided ZK Jun 9, 2025
@cristianrgreco cristianrgreco added bug Something isn't working patch Backward compatible bug fix labels Jun 9, 2025
@cristianrgreco cristianrgreco merged commit adf3cda into main Jun 9, 2025
20 checks passed
@cristianrgreco cristianrgreco deleted the fix-flaky-kafka-test branch June 9, 2025 14:28
cristianrgreco added a commit that referenced this pull request Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working patch Backward compatible bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants