dell / csi-baremetal

Bare-metal CSI Driver
Apache License 2.0
72 stars 33 forks source link

Feature storage group poc manual label drive #993

Open CraneShiEMC opened 1 year ago

CraneShiEMC commented 1 year ago

Purpose

Resolves #

Describe your changes

PR checklist

Testing

Provide test details

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 79.48% and project coverage change: +0.01 :tada:

Comparison is base (4166dbf) 72.31% compared to head (833a677) 72.32%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #993 +/- ## ========================================== + Coverage 72.31% 72.32% +0.01% ========================================== Files 62 62 Lines 8115 8140 +25 ========================================== + Hits 5868 5887 +19 - Misses 1993 1997 +4 - Partials 254 256 +2 ``` | Flag | Coverage Δ | | |---|---|---| | unittests | `72.32% <79.48%> (+0.01%)` | :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=dell#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | Coverage Δ | | |---|---|---| | [pkg/common/ac\_operations.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL2NvbW1vbi9hY19vcGVyYXRpb25zLmdv) | `8.62% <0.00%> (ø)` | | | [pkg/common/volume\_operations.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL2NvbW1vbi92b2x1bWVfb3BlcmF0aW9ucy5nbw==) | `82.16% <0.00%> (-0.50%)` | :arrow_down: | | [pkg/base/k8s/kubeclient.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL2Jhc2UvazhzL2t1YmVjbGllbnQuZ28=) | `65.69% <62.50%> (-0.41%)` | :arrow_down: | | [pkg/base/capacityplanner/node\_capacity.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL2Jhc2UvY2FwYWNpdHlwbGFubmVyL25vZGVfY2FwYWNpdHkuZ28=) | `94.54% <100.00%> (ø)` | | | [...ontroller/capacitycontroller/capacitycontroller.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL2NvbnRyb2xsZXIvY2FwYWNpdHljb250cm9sbGVyL2NhcGFjaXR5Y29udHJvbGxlci5nbw==) | `74.18% <100.00%> (+1.28%)` | :arrow_up: | | [pkg/node/volumemgr.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL25vZGUvdm9sdW1lbWdyLmdv) | `74.66% <100.00%> (ø)` | | | [pkg/scheduler/extender/extender.go](https://codecov.io/gh/dell/csi-baremetal/pull/993?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell#diff-cGtnL3NjaGVkdWxlci9leHRlbmRlci9leHRlbmRlci5nbw==) | `54.90% <100.00%> (+0.28%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.