Bug 1767542

Summary: Port IMS Workflow - VM Transform
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: V2VAssignee: Fabien Dupont <fdupont>
Status: CLOSED CURRENTRELEASE QA Contact: Nandini Chandra <nachandr>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.11.0CC: bthurber, fdupont, mshriver
Target Milestone: GAKeywords: ZStream
Target Release: 5.11.4Flags: pm-rhel: cfme-5.11.z+
simaishi: mirror+
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: 1746592 Environment:
Last Closed: 2020-03-17 13:24:00 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: 1746592    
Bug Blocks:    

Comment 2 CFME Bot 2019-10-31 21:36:01 UTC
New commit detected on ManageIQ/manageiq/ivanchuk:

https://github.com/ManageIQ/manageiq/commit/4c4117e1d33972b317fca19088807c9dc6b4a6b6
commit 4c4117e1d33972b317fca19088807c9dc6b4a6b6
Author:     Adam Grare <agrare>
AuthorDate: Thu Aug 29 11:05:28 2019 -0400
Commit:     Adam Grare <agrare>
CommitDate: Thu Aug 29 11:05:28 2019 -0400

    Merge pull request #19222 from fdupont-redhat/v2v_state_machine_transform_vm

    V2v state machine transform vm

    (cherry picked from commit 4c54666e83d6e7a44600f8a701905453dd046090)

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

 app/models/infra_conversion_job.rb | 55 +-
 spec/models/infra_conversion_job_spec.rb | 138 +-
 2 files changed, 184 insertions(+), 9 deletions(-)

Comment 3 CFME Bot 2019-10-31 21:51:46 UTC
New commit detected on ManageIQ/manageiq-content/ivanchuk:

https://github.com/ManageIQ/manageiq-content/commit/35cf51360efe3659a142dc124866cc099487e8dc
commit 35cf51360efe3659a142dc124866cc099487e8dc
Author:     Greg McCullough <gmccullo>
AuthorDate: Mon Sep  2 12:17:29 2019 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Mon Sep  2 12:17:29 2019 -0400

    Merge pull request #569 from fdupont-redhat/v2v_state_machine_transform_vm

    [V2V] Remove VMTransform and VMCheckTransformed states

    (cherry picked from commit 3b0e6b2fd0176e894d07df24b95203f696b65f32)

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

 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.rb | 59 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.yaml | 15 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.rb | 27 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.yaml | 15 -
 content/automate/ManageIQ/Transformation/StateMachines/VMTransformation.class/transform.yaml | 17 -
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed_spec.rb | 89 -
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform_spec.rb | 45 -
 7 files changed, 267 deletions(-)

Comment 4 Shveta 2019-11-21 22:06:13 UTC
Hi , 
Please provide the verification steps for this.
Thanks, 
Shveta

Comment 5 Fabien Dupont 2019-11-22 07:23:24 UTC
@shveta, you should only test non regression. We backported some code from Automate to the model, so it shouldn't change the behavior.
It is the same for all "Port IMS Workflow" BZs.

Comment 7 CFME Bot 2019-12-03 16:23:35 UTC
New commit detected on ManageIQ/manageiq-content/ivanchuk:

https://github.com/ManageIQ/manageiq-content/commit/1c76cd29d3f46017ef62310a654673b4a2cdaae0
commit 1c76cd29d3f46017ef62310a654673b4a2cdaae0
Author:     Satoe Imaishi <simaishi>
AuthorDate: Tue Dec  3 11:12:42 2019 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Dec  3 11:12:42 2019 -0500

    Revert "Merge pull request #569 from fdupont-redhat/v2v_state_machine_transform_vm"

    This reverts commit 35cf51360efe3659a142dc124866cc099487e8dc.

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

 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.rb | 59 +
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.yaml | 15 +
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.rb | 27 +
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.yaml | 15 +
 content/automate/ManageIQ/Transformation/StateMachines/VMTransformation.class/transform.yaml | 17 +
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed_spec.rb | 89 +
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform_spec.rb | 45 +
 7 files changed, 267 insertions(+)

Comment 8 CFME Bot 2019-12-03 18:10:59 UTC
New commit detected on ManageIQ/manageiq/ivanchuk:

https://github.com/ManageIQ/manageiq/commit/9ea4a80d428332ea86cc0ec3f39b5b1103ceab17
commit 9ea4a80d428332ea86cc0ec3f39b5b1103ceab17
Author:     Satoe Imaishi <simaishi>
AuthorDate: Tue Dec  3 12:45:35 2019 -0500
Commit:     Satoe Imaishi <simaishi>
CommitDate: Tue Dec  3 12:45:35 2019 -0500

    Revert "Merge pull request #19222 from fdupont-redhat/v2v_state_machine_transform_vm"

    This reverts commit 4c4117e1d33972b317fca19088807c9dc6b4a6b6.

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

 app/models/infra_conversion_job.rb | 55 +-
 spec/models/infra_conversion_job_spec.rb | 138 +-
 2 files changed, 9 insertions(+), 184 deletions(-)

Comment 9 Shveta 2020-01-21 17:52:37 UTC
Tested on upstream master.20200118225846_b8aaf77
Cold migration works

Comment 10 CFME Bot 2020-02-21 19:35:34 UTC
New commit detected on ManageIQ/manageiq/ivanchuk:

https://github.com/ManageIQ/manageiq/commit/184487b66679653f2d8a3c95caeadf692810593b
commit 184487b66679653f2d8a3c95caeadf692810593b
Author:     Adam Grare <agrare>
AuthorDate: Thu Aug 29 15:05:28 2019 +0000
Commit:     Adam Grare <agrare>
CommitDate: Thu Aug 29 15:05:28 2019 +0000

    Merge pull request #19222 from fdupont-redhat/v2v_state_machine_transform_vm

    V2v state machine transform vm

    (cherry picked from commit 4c54666e83d6e7a44600f8a701905453dd046090)

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

 app/models/infra_conversion_job.rb | 55 +-
 spec/models/infra_conversion_job_spec.rb | 138 +-
 2 files changed, 184 insertions(+), 9 deletions(-)

Comment 11 CFME Bot 2020-02-21 19:56:03 UTC
New commit detected on ManageIQ/manageiq-content/ivanchuk:

https://github.com/ManageIQ/manageiq-content/commit/ee6ec2c0b95336d55abcba89ef4827459151f879
commit ee6ec2c0b95336d55abcba89ef4827459151f879
Author:     Greg McCullough <gmccullo>
AuthorDate: Mon Sep  2 16:17:29 2019 +0000
Commit:     Greg McCullough <gmccullo>
CommitDate: Mon Sep  2 16:17:29 2019 +0000

    Merge pull request #569 from fdupont-redhat/v2v_state_machine_transform_vm

    [V2V] Remove VMTransform and VMCheckTransformed states

    (cherry picked from commit 3b0e6b2fd0176e894d07df24b95203f696b65f32)

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

 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.rb | 59 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed.yaml | 15 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.rb | 27 -
 content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform.yaml | 15 -
 content/automate/ManageIQ/Transformation/StateMachines/VMTransformation.class/transform.yaml | 17 -
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmchecktransformed_spec.rb | 89 -
 spec/content/automate/ManageIQ/Transformation/Common.class/__methods__/vmtransform_spec.rb | 45 -
 7 files changed, 267 deletions(-)

Comment 12 Nandini Chandra 2020-03-05 21:33:19 UTC
Verified that a cold migration is successful in 5.11.4.1 with UCI conversion host.