[__matrix_synapse_worker] various tweaks and fixes following testing

type/__matrix_synapse_worker/files/matrix-synapse-worker@.service

@@ -15,9 +15,9 @@ NotifyAccess=main
 User=matrix-synapse
 WorkingDirectory=/var/lib/matrix-synapse
 EnvironmentFile=/etc/default/matrix-synapse
-ExecStart=/opt/venvs/matrix-synapse/bin/python -m synapse.app.generic_worker --config-path=/etc/matrix-synapse/homeserver.yaml --config-path=/etc/matrix-synapse/conf.d/ --config-path=/etc/matrix-synapse/workers/%i.yaml
+ExecStart=/usr/bin/python3 -m synapse.app.generic_worker --config-path=/etc/matrix-synapse/homeserver.yaml --config-path=/etc/matrix-synapse/conf.d/ --config-path=/etc/matrix-synapse/workers/
 ExecReload=/bin/kill -HUP $MAINPID
-Restart=always
+Restart=on-failure
 RestartSec=3
 SyslogIdentifier=matrix-synapse-%i
 

type/__matrix_synapse_worker/files/worker.yaml.sh

@@ -6,11 +6,11 @@ worker_name: "${WORKER_NAME:?}"
 
 # The replication listener on the main synapse process.
 worker_replication_host: "${WORKER_REPLICATION_HOST:?}"
-worker_replication_http_port: "${WORKER_REPLICATION_PORT:?}"
+worker_replication_http_port: ${WORKER_REPLICATION_PORT:?}
 
 worker_listeners:
  - type: http
-   port: "${WORKER_PORT:?}"
+   port: ${WORKER_PORT:?}
    resources:
      - names:
 EOF

type/__matrix_synapse_worker/gencode-remote

@@ -0,0 +1,12 @@
+#!/bin/sh
+
+# Designed for Debian/systemd - make sure to adapt when you add support for other
+# distributions/OSs.
+name=$__object_id
+synapse_conf_dir='/etc/matrix-synapse'
+synapse_workers_conf_dir="$synapse_conf_dir/workers"
+systemd_worker_service="matrix-synapse-worker@$name"
+
+if grep -qE "^__file$synapse_workers_conf_dir/$name" "${__messages_in}"; then
+	echo "systemctl restart $systemd_worker_service"
+fi

type/__matrix_synapse_worker/manifest

@@ -8,7 +8,7 @@ os=$(cat "$__global/explorer/os")
 
 case "$os" in
 	debian)
-		synapse_conf_dir='/etc/synapse'
+		synapse_conf_dir='/etc/matrix-synapse'
 		synapse_workers_conf_dir="$synapse_conf_dir/workers"
 
 		# Synapse log configuration on debian - default value of config-log
@@ -53,14 +53,15 @@ require="__directory/$synapse_workers_conf_dir" \
 	--source "$__object/files/worker.yaml" \
 	--mode 0644
 
-__file "/etc/systemd/systemd/matrix-synapse-worker@.service" \
+__file "/etc/systemd/system/matrix-synapse-worker@.service" \
 	--source "$__type/files/matrix-synapse-worker@.service" \
 	--mode 0644 \
 	--state $systemd_worker_service_override
 
 # Start service, enable at boot.
-require="__file/$synapse_workers_conf_dir/$name.yaml \
+service_req=
-	__file/etc/systemd/systemd/matrix-synapse-worker@.service"  \
+if [ "$systemd_worker_service_override" ]; then
-	__service "$systemd_worker_service" --action start
+	service_req="__file/etc/systemd/system/matrix-synapse-worker@.service"
-require="__service/$systemd_worker_service" \
+fi
+require="__file/$synapse_workers_conf_dir/$name.yaml $service_req" \
 	__start_on_boot "$systemd_worker_service"

type/__matrix_synapse_worker/parameter/required

@@ -1,3 +1,2 @@
 app
-name
 port