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

svc 3par vnx controller add cpu count #940

Closed tanjiangyu-ghca closed 1 year ago

tanjiangyu-ghca 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 #940 (841bc42) into master (9ad2996) will decrease coverage by 0.02%. The diff coverage is 68.18%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #940 +/- ## ========================================== - Coverage 71.36% 71.33% -0.03% ========================================== Files 187 187 Lines 23067 23079 +12 Branches 3508 3510 +2 ========================================== + Hits 16462 16464 +2 - Misses 5490 5495 +5 - Partials 1115 1120 +5 ``` | [Impacted Files](https://codecov.io/gh/sodafoundation/delfin/pull/940?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation) | Coverage Δ | | |---|---|---| | [delfin/drivers/fujitsu/eternus/eternus\_stor.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvZnVqaXRzdS9ldGVybnVzL2V0ZXJudXNfc3Rvci5weQ==) | `88.88% <ø> (ø)` | | | [delfin/drivers/hpe/hpe\_3par/ssh\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvaHBlL2hwZV8zcGFyL3NzaF9oYW5kbGVyLnB5) | `78.74% <54.54%> (-1.26%)` | :arrow_down: | | [...rivers/dell\_emc/vnx/vnx\_block/component\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvZGVsbF9lbWMvdm54L3ZueF9ibG9jay9jb21wb25lbnRfaGFuZGxlci5weQ==) | `64.08% <66.66%> (+0.01%)` | :arrow_up: | | [delfin/drivers/hpe/hpe\_msa/ssh\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvaHBlL2hwZV9tc2Evc3NoX2hhbmRsZXIucHk=) | `67.28% <66.66%> (-0.01%)` | :arrow_down: | | [delfin/drivers/hpe/hpe\_3par/component\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvaHBlL2hwZV8zcGFyL2NvbXBvbmVudF9oYW5kbGVyLnB5) | `74.50% <100.00%> (+0.12%)` | :arrow_up: | | [delfin/drivers/hpe/hpe\_3par/consts.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvaHBlL2hwZV8zcGFyL2NvbnN0cy5weQ==) | `100.00% <100.00%> (ø)` | | | [delfin/drivers/ibm/storwize\_svc/ssh\_handler.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvaWJtL3N0b3J3aXplX3N2Yy9zc2hfaGFuZGxlci5weQ==) | `74.46% <100.00%> (+0.03%)` | :arrow_up: | | [delfin/drivers/fake\_storage/\_\_init\_\_.py](https://codecov.io/gh/sodafoundation/delfin/pull/940/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=sodafoundation#diff-ZGVsZmluL2RyaXZlcnMvZmFrZV9zdG9yYWdlL19faW5pdF9fLnB5) | `94.41% <0.00%> (-0.77%)` | :arrow_down: |
joseph-v commented 1 year ago

Another PR with same commits https://github.com/sodafoundation/delfin/pull/960 is merged.