Veeam: revert unnecessary VM state change if restore backup and attach to another VM

[weizhouapache]

Description

This PR reverts some changes in #7238

When restore backup of a VM and attach the volume to another VM, the original VM is not changed/touched, therefore the VM state should not be updated.

With this change, all the smoke tests for veeam (https://github.com/apache/cloudstack/blob/main/test/integration/smoke/test_backup_recovery_veeam.py) succeed.

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)
• build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

• Major
• Minor

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

How Has This Been Tested?

[weizhouapache]

@blueorangutan package

[blueorangutan]

@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c795547) 20.99% compared to head (f7d6393) 30.84%.
Report is 1 commits behind head on 4.19.

Additional details and impacted files

@@             Coverage Diff              @@
##               4.19    #8614      +/-   ##
============================================
+ Coverage     20.99%   30.84%   +9.84%     
- Complexity    19683    34083   +14400     
============================================
  Files          4849     5346     +497     
  Lines        324441   375417   +50976     
  Branches      45573    54598    +9025     
============================================
+ Hits          68121   115796   +47675     
+ Misses       246563   244325    -2238     
- Partials       9757    15296    +5539     

Flag	Coverage Δ
simulator-marvin-tests	24.71% <ø> (+3.72%)	⬆️
uitests	4.39% <ø> (?)
unit-tests	16.54% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[weizhouapache]

@blueorangutan test rocky8 vmware-70u3

[blueorangutan]

@weizhouapache a [SL] Trillian-Jenkins test job (rocky8 mgmt + vmware-70u3) has been kicked to run smoke tests

[weizhouapache]

test results

Smoke tests completed. 120 look OK, 2 have errors, 0 did not run
Only failed and skipped tests results shown below:


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_02_balanced_drs_algorithm | `Failure` | 139.25 | test_cluster_drs.py
test_02_list_cpvm_vm | `Failure` | 0.04 | test_ssvm.py
test_04_cpvm_internals | `Failure` | 0.05 | test_ssvm.py

[SadiJr]

CLGTM

[DaanHoogland]

clgtm