dell / csi-baremetal

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

[ISSUE-1157] Disk eject automatic transition from RELEASED to REMOVIN… #1158

Closed Andrzej-Zukowski closed 6 months ago

Andrzej-Zukowski commented 6 months ago

…G state

Purpose

Resolves #1157

Describe your changes

PR checklist

Testing

Provide test details

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 70.64%. Comparing base (3463762) to head (62bef15). Report is 6 commits behind head on master.

:exclamation: Current head 62bef15 differs from pull request most recent head 481eee5

Please upload reports for the commit 481eee5 to get more accurate results.

Files Patch % Lines
pkg/crcontrollers/drive/drivecontroller.go 66.66% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1158 +/- ## ========================================== - Coverage 70.65% 70.64% -0.02% ========================================== Files 105 105 Lines 10460 10497 +37 ========================================== + Hits 7391 7416 +25 - Misses 2764 2775 +11 - Partials 305 306 +1 ``` | [Flag](https://app.codecov.io/gh/dell/csi-baremetal/pull/1158/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/dell/csi-baremetal/pull/1158/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dell) | `70.64% <66.66%> (-0.02%)` | :arrow_down: | 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.

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