matevarga opened a new issue #1687: healthcheck.sh script is incorrect URL: https://github.com/apache/bookkeeper/issues/1687 **BUG REPORT** 1. Please describe the issue you observed: - What did you do? Starting a BookKeeper cluster in Docker Swarm using the official, latest Docker images. - What did you expect to see? Bookkeeper starts up. - What did you see instead? Docker Swarm kills the containers because health checks fail. Underlying issue: there's a health check script in the image at `/opt/bookkeeper/scripts/healthcheck.sh`. This has a bug: `./healthcheck.sh: line 34: /opt/bookkeeper/bin/bookkeeper shell bookiesanity: No such file or directory` Problematic code: ``` HEALTH_CHECK_CMD=${HEALTH_CHECK_CMD:-"${DEFAULT_HEALTH_CHECK_CMD}"} exec "${HEALTH_CHECK_CMD}" ``` This should be ``` HEALTH_CHECK_CMD=${HEALTH_CHECK_CMD:-"${DEFAULT_HEALTH_CHECK_CMD}"} eval "${HEALTH_CHECK_CMD}" ``` .. so `eval` instead of `exec`. **BP** This is the master ticket for tracking BP-<bp-number> : [a short description for this BP] Proposal PR - #<pr-number>
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
