Bug 1790538

Summary: Add VM validation for warm migration
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: V2VAssignee: Muhammad Arif <aarif>
Status: CLOSED ERRATA QA Contact: Md Nadeem <mnadeem>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: urgent    
Version: 5.11.0CC: aarif, bthurber, fdupont, mnadeem, sshveta
Target Milestone: GAKeywords: ZStream
Target Release: 5.11.4Flags: pm-rhel: cfme-5.11.z+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.11.4.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1760040 Environment:
Last Closed: 2020-03-17 13:11:17 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Bug Depends On: 1760040    
Bug Blocks:    
Attachments:
Description Flags
VM's with snapshots not allowed for warm migration none

Comment 3 Shveta 2020-01-16 03:10:49 UTC
Created attachment 1652618 [details]
VM's with snapshots not allowed for warm migration

Comment 4 Shveta 2020-01-16 03:11:34 UTC
Tested on upstream master.20200108224012_08891f6

Comment 5 CFME Bot 2020-02-21 21:20:37 UTC
New commits detected on ManageIQ/manageiq/ivanchuk:

https://github.com/ManageIQ/manageiq/commit/5d23be72d69cc6cccd71a84316da0a0600f3adeb
commit 5d23be72d69cc6cccd71a84316da0a0600f3adeb
Author:     Adam Grare <agrare>
AuthorDate: Tue Oct 29 19:43:00 2019 +0000
Commit:     Adam Grare <agrare>
CommitDate: Tue Oct 29 19:43:00 2019 +0000

    Merge pull request #19401 from thearifismail/bug1760040

    [V2V] Add VM validation for warm migration eligibility and updated specs to …

    (cherry picked from commit 4d3d17113bfdcdd45e7cca8b3099df9388d83f06)

    https://bugzilla.redhat.com/show_bug.cgi?id=1790538

 app/models/mixins/supports_feature_mixin.rb | 1 +
 app/models/service_template_transformation_plan/validate_config_info.rb | 1 +
 spec/models/service_template_transformation_plan_spec.rb | 43 +-
 3 files changed, 30 insertions(+), 15 deletions(-)


https://github.com/ManageIQ/manageiq/commit/79fc30adabbbc7c3493663374fdd4bd62bd5830d
commit 79fc30adabbbc7c3493663374fdd4bd62bd5830d
Author:     Martin Povolny <mpovolny>
AuthorDate: Thu Nov  7 09:41:13 2019 +0000
Commit:     Martin Povolny <mpovolny>
CommitDate: Thu Nov  7 09:41:13 2019 +0000

    Merge pull request #19467 from thearifismail/vm_warm_migrate_option

    [V2V] Added warm migration eligibility option to VM during its validation

    (cherry picked from commit 6a046848ea0dbafe1873b4b4b9f44382d5199b0d)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1790538

 app/models/transformation_mapping/vm_migration_validator.rb | 11 +-
 1 file changed, 6 insertions(+), 5 deletions(-)

Comment 6 CFME Bot 2020-02-21 22:51:04 UTC
New commit detected on ManageIQ/manageiq-providers-vmware/ivanchuk:

https://github.com/ManageIQ/manageiq-providers-vmware/commit/434a5639462bfa5801bdf06885a10eb868963651
commit 434a5639462bfa5801bdf06885a10eb868963651
Author:     Adam Grare <agrare>
AuthorDate: Tue Oct 29 19:19:53 2019 +0000
Commit:     Adam Grare <agrare>
CommitDate: Tue Oct 29 19:19:53 2019 +0000

    Merge pull request #463 from thearifismail/bug1760040

    [V2V] Added support query for warm migration

    (cherry picked from commit 9d595b87b765f39c7fa046859d53f4c8e1403c0b)

    https://bugzilla.redhat.com/show_bug.cgi?id=1790538

 app/models/manageiq/providers/vmware/infra_manager/vm.rb | 4 +
 spec/models/manageiq/providers/vmware/infra_manager/vm_spec.rb | 13 +
 2 files changed, 17 insertions(+)

Comment 7 Fabien Dupont 2020-03-03 13:47:52 UTC
To test this change, you can use the API to get the list of VM that are eligible for a given Infrastructure Mapping:

$ curl -s http://cf.example.com/api/transformation_mappings/1 -X POST -d '{ "action": "validate_vms" }' | jq
{
  "valid": [
    {
      "name": "vm1",
      "status": "ok",
      "reason": "ok",
      "cluster": "my_cluster",
      "path": "VMware/my_dc/my_folder",
      "allocated_size": "17179869184.0",
      "id": "82",
      "ems_cluster_id": "1",
      "warm_migration_compatible": true
    },
[...]
    {
      "name": "vm27",
      "status": "ok",
      "reason": "ok",
      "cluster": "my_cluster",
      "path": "VMware/my_dc",
      "allocated_size": "21474836480.0",
      "id": "16",
      "ems_cluster_id": "1",
      "warm_migration_compatible": false
    },
[...]
  ]
}

A virtual machine that has snapshot(s) will be marked as "warm_migration_compatible: false".

Comment 11 errata-xmlrpc 2020-03-17 13:11:17 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:0867