Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

31z4/zookeeper:latest is not 3.5 but 3.4 #82

Closed
JiniousChoi opened this issue Nov 5, 2019 · 6 comments
Closed

31z4/zookeeper:latest is not 3.5 but 3.4 #82

JiniousChoi opened this issue Nov 5, 2019 · 6 comments

Comments

@JiniousChoi
Copy link

JiniousChoi commented Nov 5, 2019

Before you file an issue here, please keep in mind that your issue may be not related to the image itself. Please make sure that it is, otherwise report the issue upstream.

Expected behavior

As the main README.md states at via-docker-stack-deploy-or-docker as follows

This will start Zookeeper 3.5 in replicated mode. Please note...

I expected for the given docker-compose.yml to work with a automatically-pulled lastest zookeeper but it turned out that those 2181 ports were missing.

I think the README.md itself is contradictory. One needs to either remove ;2181 part as in :2888:3888;2181 or remove This will start Zookeeper 3.5 in replicated mode. Please note... like the quote above.

Actual behavior

Tell us what happens instead.

docker pull 31z4/zookeeper:latest should fetch a version of 3.5

Steps to reproduce the behavior

As you would see below the history of the zk image reveals its version as 3.4

servers jinchoi$ docker images | grep 31z4
31z4/zookeeper                               latest                       916e61594d17        13 months ago       148MB
servers jinchoi$ docker history 31z4/zookeeper | grep 3
916e61594d17        13 months ago       /bin/sh -c #(nop)  CMD ["zkServer.sh" "start…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  ENTRYPOINT ["/docker-entr…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop) COPY file:edfd7f9189668bf2…   1.13kB              
<missing>           13 months ago       /bin/sh -c #(nop)  ENV PATH=/usr/local/sbin:…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  EXPOSE 2181 2888 3888        0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  VOLUME [/data /datalog /l…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop) WORKDIR /zookeeper-3.4.13     0B                  
<missing>           13 months ago       |2 DISTRO_NAME=zookeeper-3.4.13 GPG_KEY=C61B…   60.7MB              
<missing>           13 months ago       /bin/sh -c #(nop)  ARG DISTRO_NAME=zookeeper…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  ARG GPG_KEY=C61B346552DC5…   0B                  
<missing>           13 months ago       /bin/sh -c set -ex;     adduser -D "$ZOO_USE…   4.82kB              
<missing>           13 months ago       /bin/sh -c #(nop)  ENV ZOO_USER=zookeeper ZO…   0B                  
<missing>           13 months ago       /bin/sh -c apk add --no-cache     bash     s…   3.9MB               
<missing>           13 months ago       /bin/sh -c set -x  && apk add --no-cache   o…   78.6MB              
<missing>           13 months ago       /bin/sh -c #(nop)  ENV JAVA_ALPINE_VERSION=8…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  ENV JAVA_VERSION=8u171       0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  ENV PATH=/usr/local/sbin:…   0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  ENV JAVA_HOME=/usr/lib/jv…   0B                  
<missing>           13 months ago       /bin/sh -c {   echo '#!/bin/sh';   echo 'set…   87B                 
<missing>           13 months ago       /bin/sh -c #(nop)  ENV LANG=C.UTF-8             0B                  
<missing>           13 months ago       /bin/sh -c #(nop)  CMD ["/bin/sh"]              0B                  
<missing>           13 months ago       /bin/sh -c #(nop) ADD file:25c10b1d1b41d46a1…   4.41MB  
@eriknyk
Copy link

eriknyk commented Nov 8, 2019

same here using docker-compose:

  zookeeper:
    image: 31z4/zookeeper
    restart: always
    ports:
      - '2181:2181'
    hostname: zookeeper

runnig:

image:

app-api_zookeeper_1   31z4/zookeeper              latest   916e61594d17   141 MB

logs:

zookeeper_1        | 2019-11-08 14:56:28,149 [myid:] - INFO  [main:Environment@100] - Server environment:zookeeper.version=3.4.13-2d71af4dbe22557fda74f9a9b4309b15a7487f03, built on 06/29/2018 04:05 GMT
zookeeper_1        | 2019-11-08 14:56:28,149 [myid:] - INFO  [main:Environment@100] - Server environment:host.name=zookeeper
zookeeper_1        | 2019-11-08 14:56:28,149 [myid:] - INFO  [main:Environment@100] - Server environment:java.version=1.8.0_171
zookeeper_1        | 2019-11-08 14:56:28,149 [myid:] - INFO  [main:Environment@100] - Server environment:java.vendor=Oracle Corporation
zookeeper_1        | 2019-11-08 14:56:28,149 [myid:] - INFO  [main:Environment@100] - Server environment:java.home=/usr/lib/jvm/java-1.8-openjdk/jre

@eriknyk
Copy link

eriknyk commented Nov 8, 2019

@JiniousChoi in the meantime you can set image: 31z4/zookeeper:3.5 it will force to have installed 3.5,
this worked form me for docker-compose:

  zookeeper:
    image: '31z4/zookeeper:3.5'
    ports:
      - '2181:2181'
    hostname: zookeeper

@31z4
Copy link
Owner

31z4 commented Nov 10, 2019

Sorry, 31z4/zookeeper is no longer maintained. I updated README.md to avoid confusion.

@31z4 31z4 closed this as completed Nov 10, 2019
@JiniousChoi
Copy link
Author

@JiniousChoi in the meantime you can set image: 31z4/zookeeper:3.5 it will force to have installed 3.5,
this worked form me for docker-compose:

  zookeeper:
    image: '31z4/zookeeper:3.5'
    ports:
      - '2181:2181'
    hostname: zookeeper

Thanks and sure thing. I forced it so :) Just letting the maintainer know what he was missing.

@eriknyk
Copy link

eriknyk commented Nov 13, 2019

@JiniousChoi since it is not maintained anymore just migrate to:

  zookeeper:
    image: zookeeper
    restart: always
    ports:
      - '2181:2181'
    hostname: zookeeper

@JiniousChoi
Copy link
Author

@eriknyk Checked. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants
@eriknyk @JiniousChoi @31z4 and others