Bug 1599868

https://github.com/ManageIQ/miq_v2v_ui_plugin/issues/482

PR: https://github.com/ManageIQ/manageiq-v2v/pull/620 and https://github.com/ManageIQ/manageiq-v2v/pull/674

Moving to ON_QA as PR's are merged and ready for testing.

I noticed summery and description of current issue is little bit miss-matched.

We have ability to edit exiting migration plan (from while back) and it is working fine, but as @Brett mentioned (in description) I am assuming issue is about - 

If an infrastructure mapping is deleted its associated migration plan should be edited (before and after deletion). 

I tried following steps -
1. Created infra map 
2. Created migration plan (associated with step1 map)
3. Deleted infra map
4. Can't edit migration plan which was associated with step1 map

@Mike, can you clarify what should I expect here? If ^^ are correct then it should be failedQA.

Tested on: 5.10.0.23.20181106165157_92dd189

Created attachment 1504614 [details]
can't edit plan after map deletion

@Ytale You are correct. The original PRs did not address the use case in @Brett's description. There was a later PR that allows a user to edit a migration plan with a deleted mapping: https://github.com/ManageIQ/manageiq-v2v/pull/765 It was backported 5 days ago, so should be available to test. Sorry for the confusion.

Thanks for clarification @Vince.

Marking FailedQA as patch did not addressed the usecase (Comment #7).
Will wait and test this again in next build of stream 5.10.

@Vince, that PR only added the warning about the wizard steps being cleared when you are changing the mapping. @Ytale is actually correct, the PR referenced in this BZ makes it possible to change mappings in that wizard, but it did not re-enable the Edit button on a plan with a missing mapping. That last step never got implemented, which is my mistake.

I'll put up a quick PR to fix this asap.

Here's a PR that will resolve the rest of this BZ: https://github.com/ManageIQ/manageiq-v2v/pull/788

New commit detected on ManageIQ/manageiq-v2v/master:

https://github.com/ManageIQ/manageiq-v2v/commit/828e229770743655aff17ee17eb627e0ff38a882
commit 828e229770743655aff17ee17eb627e0ff38a882
Author:     Mike Turley <mturley>
AuthorDate: Tue Nov 13 10:24:59 2018 -0500
Commit:     Mike Turley <mturley>
CommitDate: Tue Nov 13 10:24:59 2018 -0500

    Enable editing orphaned plans and handle undefined mapping when editing

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

 app/javascript/react/screens/App/Overview/components/Migrations/MigrationsNotStartedList.js | 2 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/PlanWizardGeneralStep.js | 11 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/index.js | 5 +-
 6 files changed, 26 insertions(+), 7 deletions(-)

New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/17fc1605ed81de37de7f8a6a437b3cd8d416f0f8
commit 17fc1605ed81de37de7f8a6a437b3cd8d416f0f8
Author:     Michael Ro <mikerodev>
AuthorDate: Thu Nov 15 08:17:55 2018 -0500
Commit:     Michael Ro <mikerodev>
CommitDate: Thu Nov 15 08:17:55 2018 -0500

    Merge pull request #788 from mturley/bz1599868-edit-orphaned-plan

    Enable editing orphaned plans and handle undefined mapping when editing

    (cherry picked from commit 828e1ef5a1b8a25e55a364a05d6c512f01242ab3)

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

 app/javascript/react/screens/App/Overview/Overview.js | 51 +-
 app/javascript/react/screens/App/Overview/components/Migrations/MigrationsNotStartedList.js | 2 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardAdvancedOptionsStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/PlanWizardGeneralStep.js | 11 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardGeneralStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/index.js | 5 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/index.js | 5 +-
 app/javascript/react/screens/App/common/ConfirmModal.js | 8 +-
 8 files changed, 74 insertions(+), 18 deletions(-)

@Ytale, this should now be fully fixed in the next build.

Created attachment 1511297 [details]
fixed_proof.png

Fixed! I can edit plan and mentioned issue is also fixed with warning flash.
Verified on - 5.10.0.27.20181128170555_43ed8cb

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