Fedora Account System
Red Hat Associate
Red Hat Customer
Created attachment 1499023 [details] ScreenVideoRecording Description of problem: Watch the attached video, edit mapping yields blank page with issues in console. Version-Release number of selected component (if applicable): 5.10.0.21 How reproducible: 100% Steps to Reproduce: 1.Add VMware/RHV providers 2.Create a few infrastructure mappings 3.Remove one of the providers and add it back 4.Try to edit mappings for the infra mapping which has missing network or cluster or storage Actual results: watch the video attached, page goes blank, errors in console Expected results: Handle this situation gracefully, or disable edit mapping for these kind of infra mappings. Additional info:
https://github.com/ManageIQ/manageiq-v2v/issues/749
https://github.com/ManageIQ/manageiq-v2v/pull/806
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/2e1a75d68541fdcb0b7fe093f49051ba13ae89ee commit 2e1a75d68541fdcb0b7fe093f49051ba13ae89ee Author: Mike Ro <mikerodev> AuthorDate: Tue Nov 20 16:25:23 2018 -0500 Commit: Mike Ro <mikerodev> CommitDate: Tue Nov 20 16:25:23 2018 -0500 Disable edit button for broken mappings If any clusters/datastores/networks that make up an infra mapping are missing from the database, disable the edit button. https://bugzilla.redhat.com/show_bug.cgi?id=1644399 app/javascript/react/screens/App/Mappings/components/InfrastructureMappingsList/InfrastructureMappingsList.js | 4 +- 1 file changed, 3 insertions(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq-v2v/hammer: https://github.com/ManageIQ/manageiq-v2v/commit/430ca5fe40cd24455972dbe9411da35b3dfcf1d8 commit 430ca5fe40cd24455972dbe9411da35b3dfcf1d8 Author: Mike Turley <mike.turley.umass.edu> AuthorDate: Tue Nov 27 15:55:56 2018 -0500 Commit: Mike Turley <mike.turley.umass.edu> CommitDate: Tue Nov 27 15:55:56 2018 -0500 Merge pull request #806 from michaelkro/749-disable-edit-button [#749] Disable edit button for broken mappings (cherry picked from commit 8a5d5103eae6a8f91e4d822e81a461166cf589cc) https://bugzilla.redhat.com/show_bug.cgi?id=1644399 app/javascript/react/screens/App/Mappings/components/InfrastructureMappingsList/InfrastructureMappingsList.js | 4 +- 1 file changed, 3 insertions(+), 1 deletion(-)
Discussed related things offline. https://gitter.im/ManageIQ/v2v?at=5c0fbc9e33d9e11991b65391 With ^^ devel's fix, we are no longer supporting maps to be edited after step3, this is considered as a edge use case at the moment. Thanks a lot @Brett and @Mike for explanations.