gravity-sync icon indicating copy to clipboard operation
gravity-sync copied to clipboard

detect_remote_pihole does not respect REMOTE_DOCKER_BINARY

Open timatlee opened this issue 2 years ago • 1 comments

Issue Description Testing for the type of remote Pihole installation, I find that REMOTE_DETECT_DOCKER=$(${OS_SSH_CMD} -p ${GS_SSH_PORT} -i "${GS_SSH_PKIF}" ${REMOTE_USER}@${REMOTE_HOST} "sudo docker container ls | grep 'pihole/pihole'" 2>/dev/null) fails because the command cannot be found. Replacing the line with

REMOTE_DETECT_DOCKER=$(${OS_SSH_CMD} -p ${GS_SSH_PORT} -i "${GS_SSH_PKIF}" ${REMOTE_USER}@${REMOTE_HOST} "sudo ${REMOTE_DOCKER_BINARY} container ls | grep 'pihole/pihole'" 2>/dev/null) yields success, because for whatever unholy reason, Synology installs docker to /usr/local/bin/docker instead of the assumed /usr/bin/docker.

Configuration Details

# Pi-hole Docker/Podman container name - Docker will pattern match anything set below
LOCAL_DOCKER_CONTAINER='pihole'
REMOTE_DOCKER_CONTAINER='pihole'                                # Remote Pi-hole container name


REMOTE_PIHOLE_DIRECTORY='/volume1/docker/pihole/pihole'
REMOTE_DNSMASQ_DIRECTORY='/volume1/docker/pihole/dnsmasq.d'
REMOTE_DOCKER_BINARY='/usr/local/bin/docker'

timatlee avatar Jun 20 '22 23:06 timatlee

Good catch. I'll get this fixed.

vmstan avatar Jun 22 '22 02:06 vmstan