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

Expand monitor_process to handle multiple processes #131

Open
BrianJKoopman opened this issue Jan 12, 2024 · 1 comment
Open

Expand monitor_process to handle multiple processes #131

BrianJKoopman opened this issue Jan 12, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@BrianJKoopman
Copy link
Member

_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.

@BrianJKoopman BrianJKoopman added the enhancement New feature or request label Jan 12, 2024
@BrianJKoopman
Copy link
Member Author

There was a recent crash on satp1 (run 1658) where all pysmurf-controller agents crashed due to a networking issue, but the ACU continued to scan until the end of the scan (~45 minutes.) The schedule crashed on the next pysmurf command, but it would have been better if we were actively monitoring the streaming processes and crashed out earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant