HEPCloud / decisionengine

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

Added two metrics on the Source - de_source_status and de_source_acquire_seconds #686

Closed skylerfoster67 closed 10 months ago

skylerfoster67 commented 11 months ago

Added the two metrics I created; de_source_status displays the status of each source and de_source_acquire_seconds gives information on the source data

codecov[bot] commented 11 months ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (97915e0) 97.25% compared to head (c861eb3) 97.25%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #686 +/- ## ======================================= Coverage 97.25% 97.25% ======================================= Files 51 51 Lines 3165 3172 +7 Branches 517 518 +1 ======================================= + Hits 3078 3085 +7 Misses 54 54 Partials 33 33 ``` | Flag | Coverage Δ | | |---|---|---| | python-3.11 | `97.22% <100.00%> (+<0.01%)` | :arrow_up: | | python-3.9 | `97.09% <100.00%> (-0.09%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HEPCloud#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files Changed](https://app.codecov.io/gh/HEPCloud/decisionengine/pull/686?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HEPCloud) | Coverage Δ | | |---|---|---| | [...c/decisionengine/framework/engine/SourceWorkers.py](https://app.codecov.io/gh/HEPCloud/decisionengine/pull/686?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HEPCloud#diff-c3JjL2RlY2lzaW9uZW5naW5lL2ZyYW1ld29yay9lbmdpbmUvU291cmNlV29ya2Vycy5weQ==) | `97.86% <100.00%> (+0.08%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.