From dbf5d03807e66f15b1b661a4fac51c6739cad156 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Petr=20=C5=A0tetiar?= Date: Thu, 24 Oct 2024 19:55:04 +0000 Subject: [PATCH] buildworker: start: fix broken non TLS setups MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 --- docker/buildworker/files/start.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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" \ -- 2.30.2