search

confluentinc / kafka-rest

Confluent REST Proxy for Kafka
https://docs.confluent.io/current/kafka-rest/docs/index.html
Other
38 stars 643 forks source link

KREST-12630 Exclude logback from zookeeper dependency in tests #1221

Closed trnguyencflt closed 10 months ago

trnguyencflt commented 10 months ago

Version 3.8.3 of zookeeper from upstream makes tests hang, so use version 3.7.2 instead, this version also fixes the CVE-2023-44981

ehumber commented 10 months ago

How do we remember to eventually update the ZK level in the future? Is there a ZK fix coming?

trnguyencflt commented 10 months ago

How do we remember to eventually update the ZK level in the future? Is there a ZK fix coming?

@ehumber actually, this might raise a concern on any change to performance of clients, I've raised this kafka-eng channel.

trnguyencflt commented 10 months ago

@ehumber after a bit of dependency and log digging, it is very likely that logback is the real culprit of making the tests hang, I've made a new commit to exclude logback, let's hope the build will pass now 🤞