Skip to content

[type: test] Fix health check for zookeeper:3.8.0 in E2E#5008

Merged
moremind merged 4 commits intoapache:masterfrom
TeslaCN:test/zookeeper-health-check
Aug 15, 2023
Merged

[type: test] Fix health check for zookeeper:3.8.0 in E2E#5008
moremind merged 4 commits intoapache:masterfrom
TeslaCN:test/zookeeper-health-check

Conversation

@TeslaCN
Copy link
Copy Markdown
Member

@TeslaCN TeslaCN commented Aug 14, 2023

Related to #4934.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

There is no curl but nc in zookeeper:3.8.0.

$ docker run --rm -d zookeeper:3.8.0                   
e086edb76143feb588e9fc7d71b5dc2633f674cad1a8d2569e991119ece126d5

$ docker ps 
CONTAINER ID   IMAGE             COMMAND                  CREATED         STATUS         PORTS                                    NAMES
e086edb76143   zookeeper:3.8.0   "/docker-entrypoint.…"   3 seconds ago   Up 3 seconds   2181/tcp, 2888/tcp, 3888/tcp, 8080/tcp   compassionate_beaver

$ docker exec -i -t e086edb76143 bash
root@e086edb76143:/apache-zookeeper-3.8.0-bin# curl 
bash: curl: command not found
root@e086edb76143:/apache-zookeeper-3.8.0-bin# echo srvr | nc localhost 2181
Zookeeper version: 3.8.0-5a02a05eddb59aee6ac762f7ea82e92a68eb9c0f, built on 2022-02-25 08:49 UTC
Latency min/avg/max: 0/0.0/0
Received: 1
Sent: 0
Connections: 1
Outstanding: 0
Zxid: 0x0
Mode: standalone
Node count: 5
root@e086edb76143:/apache-zookeeper-3.8.0-bin# 

@TeslaCN TeslaCN marked this pull request as ready for review August 14, 2023 15:24
@TeslaCN TeslaCN changed the title Fix health check for zookeeper:3.8.0 [type: test] Fix health check for zookeeper:3.8.0 in e2e Aug 15, 2023
@TeslaCN TeslaCN changed the title [type: test] Fix health check for zookeeper:3.8.0 in e2e [type: test] Fix health check for zookeeper:3.8.0 in E2E Aug 15, 2023
@moremind moremind added this to the 2.6.1 milestone Aug 15, 2023
@moremind moremind merged commit bdbee45 into apache:master Aug 15, 2023
@TeslaCN TeslaCN deleted the test/zookeeper-health-check branch August 15, 2023 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants