Description of problem: While implementing the different sub-BZs of 1740510, we introduced regressions. That was expected, but we have to fix them. This BZ will aggregate all the PRs that fixes the regressions.
Remove handover from shutdown_vm transition https://github.com/ManageIQ/manageiq/pull/19371
Extend InfraConversionJob timeout https://github.com/ManageIQ/manageiq/pull/19373
Fix progress percentage calculation https://github.com/ManageIQ/manageiq/pull/19375
Related UI issues: https://github.com/ManageIQ/manageiq-v2v/issues/1045 https://github.com/ManageIQ/manageiq-v2v/issues/1046
PR to fix issue #1046: https://github.com/ManageIQ/manageiq-v2v/pull/1049
PR to fix issue #1045: https://github.com/ManageIQ/manageiq-v2v/pull/1055 Also, I opened another issue to follow up on when the status message strings are finalized, since we will need to update the UI's strings to match. https://github.com/ManageIQ/manageiq-v2v/issues/1054 Fabien, if you let me know when those are final, I will follow up with a final PR to close #1054.
@Mike, I think we're ready. CFME 5.11.1 builds have started and these BZs/PRs should get in. What's the status of https://github.com/ManageIQ/manageiq-v2v/issues/1054 ?
@Fabien, I left #1054 open just to remind us to go back and make sure the new migration status messages get translated, but I was waiting for confirmation that we have them all finalized. Are these the final messages? https://github.com/ManageIQ/manageiq/blob/master/app/models/infra_conversion_job.rb#L76-L128 Also, Milan had expressed that he thinks the API should return already-translated strings here and the UI shouldn't have to do it. But perhaps that is out of scope here. I'll let him comment on that.
Yes, issue 1054 needs to be fixed on backend side, but I haven't got around to fix it yet.
Fabien, if we're in a hurry to resolve this BZ before Milan has a chance to fix #1054, I think it would be okay to open a separate BZ for updating the translations of those strings. If we backport https://github.com/ManageIQ/manageiq-v2v/pull/1055 the strings will work fine in English, since an unknown string will just skip translation.
Fabien, we've merged https://github.com/ManageIQ/manageiq-v2v/pull/1076 to bring the new backend strings into the UI, but I'm not sure if what I used there is actually the latest list of strings, can you check it out when you get a chance?
LGTM. I needed we'll adjust later, but the workflow is merged, so the states messages shouldn't change.
We are good with this PR and this also closes the workflow refactor. Moving to POST for inclusion in 5.11.2.
https://github.com/ManageIQ/manageiq/pull/19725 https://github.com/ManageIQ/manageiq/pull/19763 https://github.com/ManageIQ/manageiq-content/pull/634 https://github.com/ManageIQ/manageiq/pull/19773 https://github.com/ManageIQ/manageiq/pull/19782 https://github.com/ManageIQ/manageiq/pull/19783