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:
Created attachment 1619591 [details] The error is present in migration plan as well
Issue to track this in the GitHub project board: https://github.com/ManageIQ/manageiq-v2v/issues/1038
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.
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.
Ok, perfect. In that case I will remove all UI logic checking for valid RSA keys in these views. Thanks Fabien
PR to fix this: https://github.com/ManageIQ/manageiq-v2v/pull/1041
The fix is now merged to master.