sodafoundation / delfin

delfin is the SODA Infrastructure Manager project which provides unified, intelligent and scalable resource management, alert and performance monitoring
https://sodafoundation.io/
Apache License 2.0
202 stars 354 forks source link

macro_san v3 compatibility and ssh close #974

Open zhilong-xu opened 1 year ago

zhilong-xu commented 1 year ago

What this PR does / why we need it:

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

codecov[bot] commented 1 year ago

Codecov Report

Merging #974 (c293f22) into master (f2765e1) will decrease coverage by 0.05%. The diff coverage is 42.64%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #974 +/- ## ========================================== - Coverage 70.83% 70.79% -0.05% ========================================== Files 195 195 Lines 23987 24048 +61 Branches 3674 3683 +9 ========================================== + Hits 16991 17024 +33 - Misses 5831 5858 +27 - Partials 1165 1166 +1 ``` | [Impacted Files](https://codecov.io/gh/sodafoundation/delfin/pull/974?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation) | Coverage Δ | | |---|---|---| | [delfin/drivers/macro\_san/ms/macro\_ssh\_client.py](https://codecov.io/gh/sodafoundation/delfin/pull/974?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvbWFjcm9fc2FuL21zL21hY3JvX3NzaF9jbGllbnQucHk=) | `16.17% <0.00%> (-0.25%)` | :arrow_down: | | [delfin/drivers/macro\_san/ms/ms\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/974?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvbWFjcm9fc2FuL21zL21zX2hhbmRsZXIucHk=) | `63.02% <40.31%> (-1.57%)` | :arrow_down: | | [delfin/drivers/macro\_san/ms/consts.py](https://codecov.io/gh/sodafoundation/delfin/pull/974?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvbWFjcm9fc2FuL21zL2NvbnN0cy5weQ==) | `100.00% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://codecov.io/gh/sodafoundation/delfin/pull/974/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation)