Bug 1958649 - Mapping information disappears from "Plan summary" page if the plan is edited
Summary: Mapping information disappears from "Plan summary" page if the plan is edited
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: User Experience
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 2.0.0
Assignee: Mike Turley
QA Contact: Igor Braginsky
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-05-09 14:55 UTC by Igor Braginsky
Modified: 2021-06-24 07:38 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-06-10 17:12:05 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Storage and network mapping fields are empty (74.20 KB, image/png)
2021-05-09 14:55 UTC, Igor Braginsky
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2021:2381 0 None None None 2021-06-10 17:12:11 UTC

Description Igor Braginsky 2021-05-09 14:55:16 UTC
Created attachment 1781295 [details]
Storage and network mapping fields are empty

Description of problem: Plan wizard summary page doesn't reflect ANY mapping info when editing existing plan


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


How reproducible: 100%


Steps to Reproduce:
1. Create a new provider, network and storage mappings and plan that will use all that
2. Edit just created plan
3. Go to summary page

Actual results: Storage and network mapping fields will be empty


Expected results: Mapping fields should be populated


Additional info:

Comment 1 Fabien Dupont 2021-05-10 08:05:49 UTC
I think this BZ is related to https://bugzilla.redhat.com/show_bug.cgi?id=1958652.
My understanding is that the wizard doesn't load the data from the mapping linked to the plan which has (at least) two side effects:

1. The default is again pre-selected when going through all the steps.
2. The review is empty.

@mturley @gdubreui can you check and tell me if I'm correct?

Comment 2 Mike Turley 2021-05-10 09:22:27 UTC
That's definitely a regression, we load the data from the linked mapping and it had been working fine recently.

Igor, can you share the cluster where you're seeing this?

Comment 3 Igor Braginsky 2021-05-10 09:30:57 UTC
This is mgn03 on PSI. Do you need connection details or is this enough for you?

Comment 4 Mike Turley 2021-05-10 16:51:20 UTC
Should be fixed by https://github.com/konveyor/forklift-ui/pull/591 (backport: https://github.com/konveyor/forklift-ui/pull/592).

See notes in the PR.. I'm not certain this will fix the issue for network mappings, only storage mappings. I was not able to reproduce the issue for network mappings. @ibragins if you can help me reproduce that, either now or when verifying this fix, that would be great.

I suspect the network mapping part of the issue may have been caused by either source or target networks being changed/removed in the providers after the plan was created and before it was edited. Opened https://github.com/konveyor/forklift-ui/issues/590 to handle that, but if that's the case I don't think it should be a release blocker.

Comment 5 Mike Turley 2021-05-10 19:25:41 UTC
Reproduced and figure out the issue with the network mappings here, it was related to cache invalidation. Also fixed in the above PR.

Comment 6 Fabien Dupont 2021-05-11 06:29:33 UTC
The fix is in build 2.0.0-13 / iib:74067.

Comment 7 Igor Braginsky 2021-05-11 09:03:36 UTC
Verified this on MTV 2.0.0-13, mapping info is on its place and it is correct

Comment 10 errata-xmlrpc 2021-06-10 17:12:05 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 (MTV 2.0.0 images), 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/RHEA-2021:2381


Note You need to log in before you can comment on or make changes to this bug.