search

HEPCloud / decisionengine

HEPCloud Decision Engine framework
Apache License 2.0
6 stars 26 forks source link

Ensure we are calling the cherrypy shutdown methods #606

Closed jcpunk closed 2 years ago

jcpunk commented 2 years ago

There may be some stuck cherrypy processes. This should help them self-timeout or cleanly shutdown.

codecov[bot] commented 2 years ago

Codecov Report

Merging #606 (baaa3f3) into master (6117cc9) will decrease coverage by 0.09%. The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #606      +/-   ##
==========================================
- Coverage   96.04%   95.95%   -0.10%     
==========================================
  Files          47       47              
  Lines        2835     2843       +8     
  Branches      449      451       +2     
==========================================
+ Hits         2723     2728       +5     
- Misses         83       85       +2     
- Partials       29       30       +1
Flag Coverage Δ
python-3.10 95.67% <50.00%> (-0.10%) :arrow_down:
python-3.6 95.54% <50.00%> (-0.10%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
.../decisionengine/framework/engine/DecisionEngine.py 87.50% <50.00%> (-0.47%) :arrow_down:
src/decisionengine/framework/engine/de_client.py 96.69% <0.00%> (+0.08%) :arrow_up:

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 6117cc9...baaa3f3. Read the comment docs.