simonsobs / simonsobs/sorunlib
Expand monitor_process to handle multiple processes
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 0
- Forks
- 0
- Avg merge
- 17h 32m
- Merged PRs (30d)
- 1
Description
_internal.monitor_process currently monitors the ACU agent's generate_scan process during a scan, and will raise if the process crashes. However there are more processes we need to monitor during scans. monitor_process should be expanded to take in a list of processes to monitor. This will include the smurf stream processes, for example.
Contributor guide
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 by locating _internal.monitor_process and its use with the ACU agent's generate_scan process. Trace how process crashes are detected during scans, then identify the smurf stream process callers that also need coverage. Done means the monitor accepts multiple processes and raises when any monitored process crashes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100