Bug 1755897

Summary: The UI warning about RSA is deprecated and not true anymore.
Product: Red Hat CloudForms Management Engine Reporter: Nenad Peric <nperic>
Component: V2VAssignee: Mike Turley <mturley>
Status: CLOSED CURRENTRELEASE QA Contact: Shveta <sshveta>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: medium    
Version: 5.11.0CC: bthurber, dmetzger, fdupont, mshriver
Target Milestone: GAKeywords: 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:
: 1767788 (view as bug list) Environment:
Last Closed: 2020-10-26 16:26:03 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Bug Depends On: 1757352    
Bug Blocks: 1767788    
Attachments:
Description Flags
The error is present in migration plan as well none

Description Nenad Peric 2019-09-26 12:27:26 UTC 
Description of problem:

When creating infrastructure mapping, during the compute mapping we would get a warning on all the projects saying:

" You must enable an RSA key pair on the OpenStack provider. You can continue to create an infrastructure mapping without the key pair, but you must enable it before running the migration plan. "

This is a false statement and should not be raised as a warning anymore. 

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

 5.11.0.26

How reproducible:

100%

Steps to Reproduce:
1. Go to infrastructure mapping
2. try creating VMWare to OSP mapping
3. get a warning...

Actual results:

Warning is raised next to the name of projects

Expected results:

No warning should be raised, RSA keypair is not an issue anymore. 


Additional info:
Comment 2 Nenad Peric 2019-09-26 14:11:15 UTC 
Created attachment 1619591 [details]
The error is present in migration plan as well
Comment 3 Mike Turley 2019-09-27 19:26:22 UTC 
Issue to track this in the GitHub project board: https://github.com/ManageIQ/manageiq-v2v/issues/1038
Comment 4 Mike Turley 2019-09-30 20:58:23 UTC 
The warning in the plans view should be fixed, for sure.

The warning in the mapping wizard, however, should only appear if there are conversion hosts in the provider that do not have authentications configured.

Fabien, is this something we still want to check in the mapping wizard (and maybe change the phrasing to make it clear that it's a conversion host that should be fixed), or is it no longer possible to have a conversion host without an authentication (we should remove this message entirely)?

Also, is it still possible for a migration to fall back on using a provider's authentication or is it 100% only possible for a conversion host's authentication to be used? I'm curious if users can run into these edge cases if they manually configure a conversion host rather than through the UI.
Comment 5 Fabien Dupont 2019-10-01 10:17:56 UTC 
The API currently doesn't prevent someone from creating a conversion host without authentication details. I've created a BZ to implement additional checks: https://bugzilla.redhat.com/show_bug.cgi?id=1757352.
We don't want to fall back to provider's authentication, so it should using conversion authentication all the time.
Comment 6 Mike Turley 2019-10-01 14:49:09 UTC 
Ok, perfect. In that case I will remove all UI logic checking for valid RSA keys in these views. Thanks Fabien
Comment 7 Mike Turley 2019-10-01 20:09:00 UTC 
PR to fix this: https://github.com/ManageIQ/manageiq-v2v/pull/1041
Comment 8 Mike Turley 2019-10-02 18:29:08 UTC 
The fix is now merged to master.