Bug 1666840

Summary: [RFE] validate infra mappings
Product: Red Hat CloudForms Management Engine Reporter: Michael Ro <micro>
Component: V2VAssignee: Muhammad Arif <aarif>
Status: CLOSED EOL QA Contact: Shveta <sshveta>
Severity: medium Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.10.0CC: aarif, bthurber, dberger, dmetzger, fdupont, jprause, mshriver, mturley, simaishi
Target Milestone: GAKeywords: RFE, TestOnly, ZStream
Target Release: 5.12.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1768517 (view as bug list) Environment:
Last Closed: 2020-03-25 18:03:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Feature
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1768517    

Description Michael Ro 2019-01-16 17:39:29 UTC
Description of problem:

Please see the "What is a Broken Mapping" section of https://github.com/ManageIQ/manageiq-v2v/issues/766

Request:

A validate_transformation_mapping(s) endpoint/action.  The back end will check each transformation_mapping_item and ensure that all of the target/destination records exist in the CF database.  If a mapping is invalid, return the missing item(s) in the response for enhanced error reporting.

- This will be used to filter invalid mappings in the plan wizard to prevent users from inadvertently creating plans with broken mappings.
- This will be used to add warnings to non-started plans whose mappings have been invalidated
- This will be used on the mappings page to flag invalid mappings and inform users which part of the mapping(s) are missing from the CF database

Comment 4 Mike Turley 2019-03-07 15:33:59 UTC
This BZ represents the backend support for part of what we discussed in https://github.com/ManageIQ/manageiq-v2v/issues/754.

Comment 7 Fabien Dupont 2019-03-28 17:01:24 UTC
Moving to 5.10.4. It was not a IMS 1.2 feature and needs extra time.

Comment 9 Fabien Dupont 2019-04-18 20:16:56 UTC
Not enough time for 5.10.4. We'll implement validation into 5.11.

Comment 11 Muhammad Arif 2019-09-03 13:22:45 UTC
Related PRs:
[V2V] Lan validation in Transformation Mapping : https://github.com/ManageIQ/manageiq/pull/19220
Explicitly pass transformation_mapping when generating mapping items: https://github.com/ManageIQ/manageiq-api/pull/664
Explicitly pass transformation_mapping when generating mapping items: https://github.com/ManageIQ/manageiq/pull/19168
[V2V] Update the transformation mapping specs: https://github.com/ManageIQ/manageiq/pull/18986

Comment 12 Muhammad Arif 2019-09-03 13:26:53 UTC
Related PRs:
[V2V] Datastore validation in Transformation Mapping : https://github.com/ManageIQ/manageiq/pull/19204
[V2V] Lan validation in Transformation Mapping : https://github.com/ManageIQ/manageiq/pull/19220
Explicitly pass transformation_mapping when generating mapping items: https://github.com/ManageIQ/manageiq-api/pull/664
Explicitly pass transformation_mapping when generating mapping items: https://github.com/ManageIQ/manageiq/pull/19168
[V2V] Update the transformation mapping specs: https://github.com/ManageIQ/manageiq/pull/18986

Comment 13 Muhammad Arif 2019-09-25 14:27:00 UTC
Work Done.

Comment 15 Muhammad Arif 2019-11-01 19:44:06 UTC
Satoe, this work is done.

Comment 16 Muhammad Arif 2019-11-01 20:46:21 UTC
[V2V] Datastore validation in Transformation Mapping : ManageIQ/manageiq#19204 (**merged**)
[V2V] Lan validation in Transformation Mapping : ManageIQ/manageiq#19220 (**merged**)
Explicitly pass transformation_mapping when generating mapping items: ManageIQ/manageiq-api#664 (**ignore**)
Explicitly pass transformation_mapping when generating mapping items: ManageIQ/manageiq#19168 (**merged**)
[V2V] Update the transformation mapping specs: ManageIQ/manageiq#18986 (**merged**)

Comment 17 Fabien Dupont 2019-11-04 12:29:00 UTC
Satoe, is Arif's comment enough for you ? Do you need more information ?

Comment 19 Satoe Imaishi 2019-11-04 15:18:52 UTC
*** Bug 1713443 has been marked as a duplicate of this bug. ***