Bug 1645168

Summary: [RFE] [Edit Migration Plan] Allow changing the associated infrastructure mapping
Product: Red Hat CloudForms Management Engine Reporter: Mike Turley <mturley>
Component: V2VAssignee: Brett Thurber <bthurber>
Status: CLOSED ERRATA QA Contact: Yadnyawalk Tale <ytale>
Severity: low Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.10.0CC: akarol, dmetzger, fdupont, kkulkarn, simaishi, smallamp
Target Milestone: GAKeywords: FutureFeature
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.10.0.23 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-02-07 23:03:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Attachments:
Description Flags
fixed_proof.png none

Description Mike Turley 2018-11-01 14:20:24 UTC
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 2 Mike Turley 2018-11-01 14:21:14 UTC
GH PR: https://github.com/ManageIQ/manageiq-v2v/pull/757

Comment 3 CFME Bot 2018-11-02 14:22:40 UTC
New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/db74305378aac0eedd58d8e95bf1295874425cb4
commit db74305378aac0eedd58d8e95bf1295874425cb4
Author:     Michael Ro <mikerodev>
AuthorDate: Fri Nov  2 08:23:29 2018 -0400
Commit:     Michael Ro <mikerodev>
CommitDate: Fri Nov  2 08:23:29 2018 -0400

    Merge pull request #757 from mturley/edit-plan-change-mapping

    [Edit Migration Plan] Allow changing the associated infrastructure mapping

    (cherry picked from commit bb6a0ce2bf27d057a5aa84d68770bbc3345e6f13)

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

 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/PlanWizardAdvancedOptionsStep.js | 3 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/index.js | 20 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/PlanWizardGeneralStep.js | 6 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStep.js | 12 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/index.js | 3 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/PlanWizardVMStep.js | 21 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/index.js | 21 +-
 7 files changed, 55 insertions(+), 31 deletions(-)

Comment 4 CFME Bot 2018-11-02 14:22:53 UTC
New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/db74305378aac0eedd58d8e95bf1295874425cb4
commit db74305378aac0eedd58d8e95bf1295874425cb4
Author:     Michael Ro <mikerodev>
AuthorDate: Fri Nov  2 08:23:29 2018 -0400
Commit:     Michael Ro <mikerodev>
CommitDate: Fri Nov  2 08:23:29 2018 -0400

    Merge pull request #757 from mturley/edit-plan-change-mapping

    [Edit Migration Plan] Allow changing the associated infrastructure mapping

    (cherry picked from commit bb6a0ce2bf27d057a5aa84d68770bbc3345e6f13)

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

 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/PlanWizardAdvancedOptionsStep.js | 3 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/index.js | 20 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/PlanWizardGeneralStep.js | 6 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStep.js | 12 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/index.js | 3 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/PlanWizardVMStep.js | 21 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/index.js | 21 +-
 7 files changed, 55 insertions(+), 31 deletions(-)

Comment 5 Yadnyawalk Tale 2018-11-09 11:43:36 UTC
Created attachment 1503599 [details]
fixed_proof.png

Fixed! I can edit and changed associated infrastructure mapping.
Verified on - 

```
    Version 5.10.0.23.20181106165157_92dd189
    Server Name EVM
    User Name Administrator
    User Role EvmRole-super_administrator
    Browser Firefox
    Browser Version 57
    Browser OS Linux

```

Thanks!

Comment 6 errata-xmlrpc 2019-02-07 23:03:56 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/RHSA-2019:0212