The GitHub Actions job "CI" on pekko-connectors-kafka.git/kafka-4-server-tests has succeeded. Run started by GitHub user pjfanning (triggered by pjfanning).
Head commit for run: 5992f7fea29dac1d65e4876a0dfa987ecbd5a651 / PJ Fanning <[email protected]> fix duplicate broker network alias breaking multi-broker KRaft quorum Motivation: The 3-broker KRaft integration tests failed in CI: the constructor-default broker-1 network alias was never removed when the broker number changed, because testcontainers' getNetworkAliases() returns a copy of the alias list, so remove() mutated a throwaway list. Every broker container then also carried the broker-1 alias, and Docker DNS resolved broker-1 round-robin across all brokers (visible in the CI container logs), which broke the controller quorum and the failover tests. Modification: withBrokerNum builds the new alias list and applies it via setNetworkAliases, which writes through to the container definition. Result: Each broker container carries exactly its own broker-<n> alias. Tests: - standalone check instantiating brokers 0..2: each container ends up with exactly its own broker-<n> alias - multi-broker KRaft covered by the CI integration-test matrix (8.3.1) References: Refs #301 Report URL: https://github.com/apache/pekko-connectors-kafka/actions/runs/33442198440 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
