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

Fix cleanup in multi-worker envs (DAS project) #2894

Closed
3 tasks done
d4vidi opened this issue Jul 14, 2021 · 1 comment
Closed
3 tasks done

Fix cleanup in multi-worker envs (DAS project) #2894

d4vidi opened this issue Jul 14, 2021 · 1 comment

Comments

@d4vidi
Copy link
Collaborator

d4vidi commented Jul 14, 2021

Description

DAS = Device Allocation Server

A rearchitecture of Detox' device allocation concern. In a nutshell - we want to have that concern either implemented as a global-scope managed server, or integrated into the existing Detox-server itself*.

The issue address the 1st phase in what could be an inception of further future improvements. It's scope is (roughly):

  • Make existing Detox server managed at global scope (rather than having distinct servers bootstrapped and killed over and over again based on workers' lifecycles).
  • Setting up this kind of a server at global-scope (either as an independent entity, or bundled into the existing Detox server). It will strictly be running at local host, for the time being, so as to avoid adding this up into a bigger, hard-to-manage change.
  • De-facto activating the server...

(@noomorph this is what I had in mind, obviously you're the one to decide the best way to move forward!)

@stale
Copy link

stale bot commented Aug 13, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you believe the issue is still relevant, please test on the latest Detox and report back.

Thank you for your contributions!

For more information on bots in this reporsitory, read this discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants