search

nils-braun / b2luigi

Task scheduling and batch running for basf2 jobs made simple
GNU General Public License v3.0
17 stars 11 forks source link

Retry getting htcondor status #134

Closed meliache closed 2 years ago

meliache commented 3 years ago

Addresses #116, some errors when getting htcondor status. A workaround might be just retrying getting the status. Opened this as a draft because I want to refactor the code to make the subprocess with retries more clean and readable. Not sure if this is needed, depend if the issue in #116 appears again to be a problem. I haven't seen it in a while.

codecov-commenter commented 3 years ago

Codecov Report

Merging #134 (b77aa3c) into main (11a8c11) will decrease coverage by 0.30%. The diff coverage is 8.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #134      +/-   ##
==========================================
- Coverage   56.23%   55.92%   -0.31%     
==========================================
  Files          23       23              
  Lines        1508     1518      +10     
==========================================
+ Hits          848      849       +1     
- Misses        660      669       +9
Impacted Files Coverage Δ
b2luigi/batch/processes/htcondor.py 46.40% <8.33%> (-3.17%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 11a8c11...b77aa3c. Read the comment docs.