feat(run_strategy): nodes for random strategy for otp app

quantum-elixir / quantum-core

:watch: Cron-like job scheduler for Elixir

https://hexdocs.pm/quantum/

Apache License 2.0

2.28k stars 148 forks source link

Closed marinakr closed 4 months ago

marinakr commented 4 months ago

Resolves #605

maennchen commented 4 months ago

There's a few to adapt for the PR:

Checking if an application runs on the node is not enough. We'll have to see if the task supervisor is up: https://github.com/quantum-elixir/quantum-core/blob/748d944950a64300b490fb3db2e2f241e9474297/lib/quantum/supervisor.ex#L94
Theoretically a user can run multiple schedulers. Therefore we can't setup the config via the quantum config namespace.
All strategies need to implement this, not only the random one.

maennchen commented 4 months ago

To do this properly, we'll probably have to pass another argument to the NodeList.nodes function that specifies the task supervisor:

marinakr commented 4 months ago

Close this PR for now - implemented custom strategy