From: Petr Štetiar Date: Thu, 24 Oct 2024 19:55:04 +0000 (+0000) Subject: buildworker: start: fix broken non TLS setups X-Git-Tag: v21~4 X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=dbf5d03807e66f15b1b661a4fac51c6739cad156;p=buildbot.git buildworker: start: fix broken non TLS setups Changes in commit 3812ff7bb296 ("buildworker: start: fix worker startup failure after update") broke non TLS setups. So lets fix it by setting SSL: only if BUILDWORKER_TLS is set. Fixes: 3812ff7bb296 ("buildworker: start: fix worker startup failure after update") Reported-by: Paul Spooren Signed-off-by: Petr Štetiar --- diff --git a/docker/buildworker/files/start.sh b/docker/buildworker/files/start.sh index 5e0147a..0f50c50 100644 --- a/docker/buildworker/files/start.sh +++ b/docker/buildworker/files/start.sh @@ -21,7 +21,7 @@ rm -f /builder/buildbot.tac /opt/venv/bin/buildbot-worker create-worker \ --force \ --umask="0o22" \ - --connection-string="SSL:$BUILDWORKER_MASTER" \ + ${BUILDWORKER_TLS:+--connection-string="SSL:$BUILDWORKER_MASTER"} \ /builder \ "$BUILDWORKER_MASTER" \ "$BUILDWORKER_NAME" \