summary refs log tree commit diff
path: root/synapse/app/synctl.py
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2018-01-11 11:15:20 +0000
committerErik Johnston <erik@matrix.org>2018-01-11 11:37:39 +0000
commit7cd34512d850fce57bdf1dcfb3f61d5315baf639 (patch)
tree29376ebd4014e07dfd7932c7a286d5dd09d2fe1c /synapse/app/synctl.py
parentMerge branch 'master' of github.com:matrix-org/synapse into develop (diff)
downloadsynapse-7cd34512d850fce57bdf1dcfb3f61d5315baf639.tar.xz
When using synctl with workers, don't start the main synapse automatically
Diffstat (limited to '')
-rwxr-xr-xsynapse/app/synctl.py34
1 files changed, 26 insertions, 8 deletions
diff --git a/synapse/app/synctl.py b/synapse/app/synctl.py
index 3bd7ef7bba..049956746e 100755
--- a/synapse/app/synctl.py
+++ b/synapse/app/synctl.py
@@ -184,6 +184,9 @@ def main():
         worker_configfiles.append(worker_configfile)
 
     if options.all_processes:
+        # To start the main synapse with -a you need to add a worker file
+        # with worker_app == "synapse.app.homeserver"
+        start_stop_synapse = False
         worker_configdir = options.all_processes
         if not os.path.isdir(worker_configdir):
             write(
@@ -200,14 +203,29 @@ def main():
         with open(worker_configfile) as stream:
             worker_config = yaml.load(stream)
         worker_app = worker_config["worker_app"]
-        worker_pidfile = worker_config["worker_pid_file"]
-        worker_daemonize = worker_config["worker_daemonize"]
-        assert worker_daemonize, "In config %r: expected '%s' to be True" % (
-            worker_configfile, "worker_daemonize")
-        worker_cache_factor = worker_config.get("synctl_cache_factor")
-        workers.append(Worker(
-            worker_app, worker_configfile, worker_pidfile, worker_cache_factor,
-        ))
+        if worker_app == "synapse.app.homeserver":
+            # We need to special case all of this to pick up options that may
+            # be set in the main config file or in this worker config file.
+            worker_pidfile = (
+                worker_config.get("worker_pid_file")
+                or worker_config("pid_file")
+                or pidfile
+            )
+            workers.append(Worker(
+                "synapse.app.homeserver",
+                worker_configfile,
+                worker_pidfile,
+                worker_config.get("synctl_cache_factor") or cache_factor,
+            ))
+        else:
+            worker_pidfile = worker_config["worker_pid_file"]
+            worker_daemonize = worker_config["worker_daemonize"]
+            assert worker_daemonize, "In config %r: expected '%s' to be True" % (
+                worker_configfile, "worker_daemonize")
+            worker_cache_factor = worker_config.get("synctl_cache_factor")
+            workers.append(Worker(
+                worker_app, worker_configfile, worker_pidfile, worker_cache_factor,
+            ))
 
     action = options.action