HEPCloud / decisionengine

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

Add some more logging to try and trace startup state #598

Closed jcpunk closed 2 years ago

jcpunk commented 2 years ago

Some unit tests in #568 are erroring out during startup. This should help get more logging in place so we can get a better picture why.

codecov[bot] commented 2 years ago

Codecov Report

Merging #598 (e23c14e) into master (dbd49a6) will increase coverage by 0.01%. The diff coverage is 100.00%.

:exclamation: Current head e23c14e differs from pull request most recent head a576a07. Consider uploading reports for the commit a576a07 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master     #598      +/-   ##
==========================================
+ Coverage   95.96%   95.97%   +0.01%     
==========================================
  Files          46       46              
  Lines        2749     2759      +10     
  Branches      439      439              
==========================================
+ Hits         2638     2648      +10     
  Misses         77       77              
  Partials       34       34              
Flag Coverage Δ
python-3.10 95.86% <100.00%> (+0.01%) :arrow_up:
python-3.6 95.66% <100.00%> (+0.08%) :arrow_up:

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

Impacted Files Coverage Δ
.../decisionengine/framework/engine/DecisionEngine.py 90.02% <100.00%> (+0.21%) :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 dbd49a6...a576a07. Read the comment docs.

jcpunk commented 2 years ago

Thanks Lisa! I thought up a few more places we might be getting stuck and updated the PR.