xenserver / status-report

Program that gathers data for xenserver host diagnostics
GNU Lesser General Public License v2.1
1 stars 9 forks source link

CA-388587: Fix filter_xapi_clusterd_db() to capture the db and log any errors that occured. #58

Closed bernhardkaindl closed 7 months ago

bernhardkaindl commented 7 months ago

Obsoleted by #67.

While identical to #67, I've opened #67 for a clean PR history for review:

codecov-commenter commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (eb9a639) 85.55% compared to head (1027cf4) 86.63%. Report is 2 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #58 +/- ## ========================================== + Coverage 85.55% 86.63% +1.08% ========================================== Files 16 17 +1 Lines 2021 2125 +104 ========================================== + Hits 1729 1841 +112 + Misses 292 284 -8 ``` | [Flag](https://app.codecov.io/gh/xenserver/status-report/pull/58/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xenserver) | Coverage Δ | | |---|---|---| | [python2.7](https://app.codecov.io/gh/xenserver/status-report/pull/58/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xenserver) | `85.15% <100.00%> (+1.09%)` | :arrow_up: | | [python3.10.13](https://app.codecov.io/gh/xenserver/status-report/pull/58/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=xenserver) | `85.93% <100.00%> (+1.21%)` | :arrow_up: | 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=xenserver#carryforward-flags-in-the-pull-request-comment) to find out more.

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

bernhardkaindl commented 7 months ago

Obsoleted by #67.

While identical to #67, I've opened #67 for a clean PR history for review: