Improve handling of errors
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 0
- Forks
- 4
- PR merge metrics
- No merged PRs in 30d
Description
During a recent test of the Prefect workflows, a file was not available on host running the workflows due to Lustre mounting of data drives not being complete. This did not result in an Exception and a failed state in the Prefect workflow - it was shown as complete (succeeded without any errors).
This is due to incorrect handling of exceptions in linker.py - in particular, in line 43, while an error is shown to have happened, the failed state is noted but never displayed to the users as part of a failed workflow.
This lack of feedback to Prefect can be remedied by at least two different mechanisms:
- allowing exceptions to propagate upwards, so that the stack trace can be seen by Prefect
- continue using the existing "failed" mechanism and return a failed state with all of the failures in this task
The latter is preferred because allowing the exceptions to propagate will result in feedback about only the first exception seen, while the failed list enables a view of all problem files.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in linker.py at line 43 and reproduce the missing-file condition during a Prefect workflow run. Trace how the existing failed mechanism is passed back to Prefect, then verify that failures from all problem files produce a visibly failed workflow rather than a succeeded state.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100