I'm not sure about installation. Maybe it would be better to create a separate PR.

Target: replace zookeeper in integration tests.

(Without great efforts)

Only six tests failed! Nice!

And most of them because of incorrect IP protocol (need to make keeper config pluggable <listen_host>::</listen_host> in IPv6 compose env and <listen_host>0.0.0.0</listen_host> in IPv4 net)

Target: replace zookeeper in integration tests.

(Without great efforts)

Cool! BTW it's interestion if it will also work for Kafka (integration tests setup separate zookeeper for Kafka).

It will work with Kafka unless it is using ACL.
ACL will also be implemented nevertheless.

test_rename_column/test.py::test_rename_with_parallel_merges strange failure with thread sanitizer, but possibly it's OK, because Keeper also built with thread sanitizer. Let's try one more time with more verbose logs. In test_zookeeper_config/test.py::test_identity we still need to use original ZooKeeper :(

test_rename_column/test.py::test_rename_with_parallel_merges -- failed again, need to add retries to this test.

Hmmm, even no session expired messages:

$ test_rename_column/_instances$ ag 'Session expired' | wc -l
0
$ test_rename_column/_instances$ ag 'Connection loss' | wc -l
0

There are no strange errors in the clickhouse-server's log https://gist.github.com/alesapin/20ca2b5f492d8c3702e82c10bf0d6b4b. So the test was just too slow? I'll make it more lightweight.

Stress test (undefined) — Logical error thrown (see clickhouse-server.log) wow

2021.05.17 13:43:17.010805 [ 327 ] {} <Error> test_27.dst_8 (ReplicatedMergeTreeQueue): Code: 49, e.displayText() = DB::Exception: Part 2_69_79_2 intersects previous part 2_0_72_999999999. It is a bug., Stack trace (when copying this message, always include the lines below):

Seems like #23997 didn't completely helped.

test_rename_column/test.py::test_rename_with_parallel_merges yaaarrrr

I'm curious why there are so many likes?) We still have to implement data import from ZooKeeper. But it should be OK for fresh installations if you don't use zookeeper auth (quite weird thing).

I'm curious why there are so many likes?) We still have to implement data import from ZooKeeper. But it should be OK for fresh installations if you don't use zookeeper auth (quite weird thing).

Not sure about everyone else, but I've been watching your progress closely since #15090 and am really looking forward to using this :)

All failures are not related to changes