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 355 forks source link

macro_san storage serial_number #954

Closed zhilong-xu closed 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 #954 (1a089db) into master (4ba8e59) will decrease coverage by 0.01%. The diff coverage is 50.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #954 +/- ## ========================================== - Coverage 71.35% 71.34% -0.02% ========================================== Files 187 187 Lines 23067 23067 Branches 3508 3508 ========================================== - Hits 16460 16457 -3 - Misses 5491 5492 +1 - Partials 1116 1118 +2 ``` | [Impacted Files](https://codecov.io/gh/sodafoundation/delfin/pull/954?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/ms\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/954/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvbWFjcm9fc2FuL21zL21zX2hhbmRsZXIucHk=) | `65.08% <50.00%> (ø)` | | | [delfin/drivers/fake\_storage/\_\_init\_\_.py](https://codecov.io/gh/sodafoundation/delfin/pull/954/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvZmFrZV9zdG9yYWdlL19faW5pdF9fLnB5) | `93.90% <0.00%> (-0.77%)` | :arrow_down: |
joseph-v commented 1 year ago

This commit is merged in PR: https://github.com/sodafoundation/delfin/pull/959