ryanwinchester
Oban await_signal with a Workflow.add_many?
Does the Worker.await_signal work with Workflow.add_many?
I’d like to wait for every job in the add_many to complete (including awaiting_for_signal) before the next workflow step runs.
Marked As Solved
sorentwo
Reproduced the issue synthetically and applied a patch for the add_many + apply_graft case. That will be in the next patch, but the alternate version I suggested above should achieve the same result.
Also Liked
sorentwo
The add_many function builds a sub-workflow which the next step depends on. The fact that one or more of those jobs are awaiting a signal shouldn’t change dependency resolution for the rest of the workflow. Can you share a minimal example of your workflow and how you’re testing/verifying it?
sorentwo
The workflows look straightforward. Before I try to reproduce with a test case, how are you verifying it? In development, with an automated test? If it’s with a test, can you share what the test looks like?
sorentwo
This is most likely because you’re scoping the signal to the parent workflow, and add_many creates a sub workflow. If you use the sub_workflow_id instead it should find the correct dependency. However, this seems like it should work intuitively and we’ll fix it in a patch.
Last Post!
AstonJ
Popular in Questions
Other popular topics
Latest Oban Threads
Categories:
Sub Categories:
Forums
Popular Tags
- #ecto
- #liveview
- #troubleshooting
- #learning-elixir
- #deployment
- #library
- #erlang
- #testing
- #genserver
- #mix
- #absinthe
- #remote-other
- #otp
- #plug
- #how-to-question
- #macros
- #postgres
- #channels
- #elixirconf
- #exunit
- #discussion
- #code-sync
- #javascript
- #podcasts
- #onsite
- #dialyzer
- #docker
- #authentication
- #umbrella
- #full-time-contract
- #podcasts-by-brainlid
- #ecto-query
- #elixir-ls
- #phoenix_html
- #iex
- #blog-post
- #graphql
- #genstage
- #ai
- #websockets
- #supervisor
- #elixirconf-us
- #advent-of-code
- #distillery
- #processes
- #forms
- #api
- #metaprogramming
- #hex
- #security









