Current tooltip (see attachment) uses passive voice. Suggested rewrite: "You must enable at least one conversion host in the cluster. You can continue to create an infrastructure mapping that includes the target cluster, but you must enable a conversion host before running the migration plan."
Created attachment 1508501 [details] Infrastructure Mapping warning icon tooltip
No GitHub issue was ever created for this so we weren't aware of it. I just created: https://github.com/ManageIQ/manageiq-v2v/issues/830
https://github.com/ManageIQ/manageiq-v2v/pull/831
Change has been merged. See above.
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/96967b98e05fc3bfae167781dc25d1724e0d98c0 commit 96967b98e05fc3bfae167781dc25d1724e0d98c0 Author: Mike Turley <mturley> AuthorDate: Mon Dec 10 14:02:29 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Mon Dec 10 14:02:29 2018 -0500 [#830] Remove passive voice in infra mapping conversion host warning https://bugzilla.redhat.com/show_bug.cgi?id=1653169 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/components/ClustersStepForm/ClustersStepForm.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/200817bb7da14d7e2787ca6f51f94235bb3a6a0b commit 200817bb7da14d7e2787ca6f51f94235bb3a6a0b Author: Mike Turley <mturley> AuthorDate: Mon Dec 10 14:38:11 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Mon Dec 10 14:38:11 2018 -0500 Remove passive voice in OSP-specific conversion host warning https://bugzilla.redhat.com/show_bug.cgi?id=1653169 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStep.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits detected on ManageIQ/manageiq-v2v/hammer: https://github.com/ManageIQ/manageiq-v2v/commit/79923f130aa0c58b9a93f3869738918b6972baf3 commit 79923f130aa0c58b9a93f3869738918b6972baf3 Author: Michael Ro <mikerodev> AuthorDate: Mon Dec 10 14:13:59 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Mon Dec 10 14:13:59 2018 -0500 Merge pull request #831 from mturley/830-mapping-warning-passive-voice [#830] Remove passive voice in infra mapping conversion host warning (cherry picked from commit 9df1154f7f6bad4afefdc7755e21c380e4f99a3b) https://bugzilla.redhat.com/show_bug.cgi?id=1653169 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/components/ClustersStepForm/ClustersStepForm.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-v2v/commit/bd863e1b2ba5a0426b8b1e7725395e324ce59ffd commit bd863e1b2ba5a0426b8b1e7725395e324ce59ffd Author: Michael Ro <mikerodev> AuthorDate: Mon Dec 10 14:47:39 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Mon Dec 10 14:47:39 2018 -0500 Merge pull request #832 from mturley/osp-warning-passive-voice Remove passive voice in OSP-specific conversion host warning (cherry picked from commit e0cb5ad5099cdfd994e268c6b95c6099ae8903b1) https://bugzilla.redhat.com/show_bug.cgi?id=1653169 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStep.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Verified on - 5.10.0.29.20181211195209_fd990a9 Thanks to Mike who coordinated while verification.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2019:0212