Skip to content
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

run -local mode doesn't kill all spawned processes when canceled #94

Open
olgabot opened this issue Nov 29, 2018 · 4 comments
Open

run -local mode doesn't kill all spawned processes when canceled #94

olgabot opened this issue Nov 29, 2018 · 4 comments
Assignees

Comments

@olgabot
Copy link
Contributor

olgabot commented Nov 29, 2018

Hello,
I killed a reflow run -local job with Ctrl-C and a bunch of the processes are still running so I have to manually kill them. Is there a way to get Reflow to look for its spawned processes?
Warmest,
Olga

@mariusae
Copy link
Collaborator

Since -local still uses Docker, the spawned processes aren't children of Reflow and thus are not killed by the operating system when Reflow is killed. One interesting thing: if you kill Reflow like this, and then restart it, you'll find that it'll just attach itself to already running Docker executions, so that they do not need to be restarted.

We have discussed changing -local mode to use a reflowlet (Reflow server) as well: so when you run reflow -local, it would look for an existing reflow server instance or start one anew. In this case, when you kill the original reflow the reflowlet would notice that it's gone and then dispose of the running tasks.

cc: @swami-m since this is also interesting for another change he's been working on

@olgabot
Copy link
Contributor Author

olgabot commented Nov 30, 2018

Ah I see, that explains some of the issues I was having! I'd Ctrl-C the reflow process, git pull for the latest workflow and then get the exact same error, and only by running on a fresh EC2 instance could I get the configurations to refresh.

@mariusae
Copy link
Collaborator

you can also clear the local state by clearing /tmp/flow or using a different local dir: reflow run -local -localdir=/tmp/xxx ...

@mariusae
Copy link
Collaborator

This is by design currently, but we are working on changing the way local mode works so that this will not happen.

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

No branches or pull requests

3 participants