-
Notifications
You must be signed in to change notification settings - Fork 59
Start 'dendron' processes directly #945
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
2a5f57e
to
c99a153
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After this PR, does pydron.py
need to exist anymore? (It's also referenced in scripts/synapse_sytest.sh
)
550ac4e
to
cb30fe3
Compare
af3b79c
to
d3fa8ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's quite a lot changing at once here, but I think it's ok.
@@ -75,11 +75,13 @@ sub _start_process | |||
if exists $params{$_}; | |||
} | |||
|
|||
my $proc_name = $params{name} // "server"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you document this in the Pod?
Co-authored-by: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
#945 removed the pydron script, but apparently failed to remove it from the dockerfile.
Instead of using a python script to run multiple synapse workers we may as well do that in sytest. This just makes things easier to extend.
Built on top of #943