Bug 1666840 - [RFE] validate infra mappings
Summary: [RFE] validate infra mappings
Keywords:
Status: CLOSED EOL
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: V2V
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.12.0
Assignee: Muhammad Arif
QA Contact: Shveta
Red Hat CloudForms Documentation
URL:
Whiteboard:
: 1713443 (view as bug list)
Depends On:
Blocks: 1768517
TreeView+ depends on / blocked
 
Reported: 2019-01-16 17:39 UTC by Michael Ro
Modified: 2020-03-25 18:03 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1768517 (view as bug list)
Environment:
Last Closed: 2020-03-25 18:03:26 UTC
Category: Feature
Cloudforms Team: V2V
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github ManageIQ manageiq-v2v issues 754 0 'None' closed [Infra Mappings] Provide detailed info regarding why clusters/datastores/networks are missing 2020-03-19 19:25:47 UTC
Github ManageIQ manageiq-v2v issues 766 0 'None' closed [Plan Wizard/Plan ListItems] Handle plans with broken infra mappings 2020-03-19 19:25:47 UTC

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. ***


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