Skip to content

Conversation

erikjohnston
Copy link
Member

@erikjohnston erikjohnston commented Sep 1, 2020

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

@erikjohnston erikjohnston force-pushed the erikj/remove_pydron branch 2 times, most recently from 2a5f57e to c99a153 Compare September 1, 2020 14:58
Copy link
Member

@clokep clokep left a 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)

Copy link
Member

@richvdh richvdh left a 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";
Copy link
Member

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>
@erikjohnston erikjohnston merged commit 2f01f3f into develop Oct 6, 2020
@erikjohnston erikjohnston deleted the erikj/remove_pydron branch October 6, 2020 13:55
richvdh added a commit that referenced this pull request Feb 22, 2021
#945 removed the pydron script, but apparently failed to remove it from the dockerfile.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants