Migrates zipkin-kafka docker image

[codefromthecrypt]

See openzipkin-attic/docker-zipkin#86
cc @jeqo

[codefromthecrypt]

good idea

…

On Thu, Oct 17, 2019, 11:59 AM Anuraag Agrawal ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In docker/hooks/post_build <#2852 (comment)>: > docker build --build-arg version="$SOURCE_BRANCH" -f "$DOCKERFILE_PATH" -t "openzipkin/zipkin-ui:${TAGS[0]}" --target zipkin-ui . for tag in ${TAGS[@]:1}; do docker tag "openzipkin/zipkin-ui:${TAGS[0]}" "openzipkin/zipkin-ui:$tag" done +echo Building zipkin-kafka +docker build --build-arg version="$SOURCE_BRANCH" -f "docker/kafka/Dockerfile"-t "openzipkin/zipkin-kafka:${TAGS[0]}" --target zipkin-kafka . +for tag in ${TAGS[@]:1}; do + docker tag "openzipkin/zipkin-kafka:${TAGS[0]}" "openzipkin/zipkin-kafka:$tag" +done + # We also build storage images to correspond with the server version to keep schemas up to date for storage in cassandra; do Should be able to just add kafka here I think ------------------------------ In docker/README.md <#2852 (comment)>: > @@ -22,6 +22,20 @@ nginx. To build, run something like $ docker build -t openzipkin/zipkin-ui:test -f docker/Dockerfile --target zipkin-ui . ``` +## zipkin-kafka Docker image Can you either move the Cassandra readme here or this to the subdirectory? Either is fine if consistent — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2852?email_source=notifications&email_token=AAAPVV6G5TOPWCUOUH3WCY3QO7PK3A5CNFSM4JBTZSMKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCIHUTSI#pullrequestreview-302991817>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAPVV5EVCT3HOB3AHYYLWDQO7PK3ANCNFSM4JBTZSMA> .