Oren Cohen

`tests/utils.go`: Move `CreateHostDiskImage` function to `storage` package

5

**Before this PR:** `CreateHostDiskImage()` is located in `tests/utils.go`. **After this PR:** `CreateHostDiskImage()` is located in `tests/storage` package where it is more suitable to be. ### What this PR does Before...

tests, refactor: remove `NewRandomVMWithDataVolumeWithRegistryImport`

7

use `libvmi` and `libdv` directly to create VMs/VMIs whose DataVolumes are configured with registry import, and completely remove the `NewRandomVMWithDataVolumeWithRegistryImport()` function from `tests/utils.go`. ### What this PR does Before this...

monitoring: Fix wrong calculation for `KubeVirtVMIExcessiveMigrations` in an edge case

6

### What this PR does There is an edge case, usually during an upgrade, in which two or more different virt-controllers are reporting the `kubevirt_vmi_migration_succeeded` metric for the same migration...

tests/operator/operator.go: Replace `Update()` with `Patch()`

3

Replace `Update()` with `Patch()` operation in order to mitigate conflicts on updating the Kubevirt resource at the `test VirtualMachineInstancesPerNode` test case at `tests/operator/operator.go` ### What this PR does Before this...

Code Cleanup: Remove unused functions

6

5 functions have been found that nothing is calling them, so we can safely remove them from the code base. /kind cleanup ### What this PR does Before this PR:...