Skip to content

feat: run_probes - add timeout for each check

Jiří Prokop requested to merge prokop/STR-1246/run_probes_timeout into main

Description

Added timeout to each check's subprocess.

BREAKING CHANGE: config structure change + new required option

How to test

Upload the script to id-test to /usr/local/lib/python3.9/dist-packages/perun/proxy/utils/ and run it. The config should already exist.

Author's checklist

  • I have followed the contribution guidelines
  • This MR has been tested or does not change functionality
  • I have added relevant merge request dependencies (if this MR has any)
  • I have added the correct labels
  • I have assigned reviewers (if any are relevant)
  • I have edited the documentation (if the changes require it) or I have noted the need for the change if I do not have access to the documentation
  • I have marked all introduced BREAKING CHANGES or necessary DEPLOYMENT NOTES in the commit message(s)

Reviewer's checklist

  • This MR has been tested or does not change functionality
  • This MR has correct commit message format

Other information

I am not entirely sure how to proceed with the implementation of 'timeout length is configurable param'(written in task description) - should each check have it's own timeout in config or is that supposed to be a global argument for the whole script?

Each check has it's own optional timeout setting. Otherwise global default is used.

Related issues

closes STR-1246

Edited by Jiří Prokop

Merge request reports