[BUG] Can't delete Storage disk

[abonillabeeche]

Describe the bug
Deleting a Storage device from the UI doesn't work. I had to go into Longhorn and delete the disk after eviction/scheduling is disabled

To Reproduce
Hosts -> Edit Config -> Storage -> X on the Disk to remove -> Save
Hosts -> Edit Config -> Storage - Disk is still there.

Refer to https://suse.slack.com/archives/C02CN3TT61M/p1714679403477409

Expected behavior
Disk should be removed. Instead one has to go into Longhorn and remove the disk.

Environment

• Harvester ISO version: 1.3.0
• Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): R620

[harvesterhci-io-github-bot]

added backport-needed/1.3.1 issue: #5778.

[harvesterhci-io-github-bot]

added backport-needed/1.2.3 issue: #5779.

[harvesterhci-io-github-bot]

Pre Ready-For-Testing Checklist

* [ ] If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
The HEP PR is at:

• Where is the reproduce steps/test steps documented?
  The reproduce steps/test steps are at:

Test Plan: please refer to harvester/node-disk-manager#102

• Is there a workaround for the issue? If so, where is it documented?
  The workaround is at:

Remove this disk on LH UI directly

• Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
  The PR is at: controller: improve the mechanism of disk removing node-disk-manager#102

* [ ] If labeled: area/ui Has the UI issue filed or ready to be merged?
The UI issue/PR is at:

* [ ] If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
The documentation/KB PR is at:

* [ ] If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?
- The automation skeleton PR is at:
- The automation test case PR is at:

* [ ] If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
The compatibility issue is filed at:

[harvesterhci-io-github-bot]

Automation e2e test issue: harvester/tests#1261

[irishgordo]

@Vicente-Cheng -thanks for this 😄 👍 - this looks good on Harvester Version: master-9c0a3a1-head

On a bare-metal HP DL160, 3 Node Cluster, the disk can be removed physically from the node and then cleaned up appropriately on Harvester & Longhorn UI

Disk was provisioned and schedulable, prior - via extra storage class etc.