Bug 1643558

Summary: [v2v] No conversion host hostname on details page popover
Product: Red Hat CloudForms Management Engine Reporter: Yadnyawalk Tale <ytale>
Component: V2VAssignee: Brett Thurber <bthurber>
Status: CLOSED CURRENTRELEASE QA Contact: Yadnyawalk Tale <ytale>
Severity: medium Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.10.0CC: bthurber, fdupont, jprause, kkulkarn, mfeifer, simaishi, smallamp, ytale
Target Milestone: GAKeywords: Regression
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: v2v
Fixed In Version: 5.10.0.25 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-02-12 16:50:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: V2V Target Upstream Version:
Embargoed:
Bug Depends On: 1643938    
Bug Blocks:    
Attachments:
Description Flags
conversation_host_popover.png
none
popover.png none

Description Yadnyawalk Tale 2018-10-26 14:25:19 UTC
Created attachment 1497814 [details]
conversation_host_popover.png

Description of problem:
No conversion host hostname on details page popover

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

How reproducible:
100%

Comment 6 CFME Bot 2018-11-14 19:07:47 UTC
New commit detected on ManageIQ/manageiq-v2v/master:

https://github.com/ManageIQ/manageiq-v2v/commit/ed3e2a67b9352be1da5a724ef15baabcc8cba21c
commit ed3e2a67b9352be1da5a724ef15baabcc8cba21c
Author:     Mike Ro <mikerodev>
AuthorDate: Wed Nov 14 11:10:37 2018 -0500
Commit:     Mike Ro <mikerodev>
CommitDate: Wed Nov 14 11:10:37 2018 -0500

    Fetch and display conversion host name

    * Since there is not yet a way for us to fetch conversion hosts directly,
      we can get them through the RequestTask for now.
    * The action for fetching a RequestTask is decoupled from that for
      conversion_hosts so that it may be reused
    * Later when the conversion_hosts API is ready, we can update the
      action/reducer implementation

    https://bugzilla.redhat.com/show_bug.cgi?id=1643558

 app/javascript/react/screens/App/Plan/Plan.js | 13 +-
 app/javascript/react/screens/App/Plan/PlanActions.js | 36 +-
 app/javascript/react/screens/App/Plan/PlanConstants.js | 4 +
 app/javascript/react/screens/App/Plan/PlanReducer.js | 31 +-
 app/javascript/react/screens/App/Plan/components/PlanRequestDetailList/PlanRequestDetailList.js | 20 +-
 5 files changed, 94 insertions(+), 10 deletions(-)

Comment 8 CFME Bot 2018-11-15 19:22:48 UTC
New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/293dae71f421a3b6043c0cb922016a5a9cbabb1c
commit 293dae71f421a3b6043c0cb922016a5a9cbabb1c
Author:     Aparna Karve <aparna.karve>
AuthorDate: Wed Nov 14 14:06:19 2018 -0500
Commit:     Aparna Karve <aparna.karve>
CommitDate: Wed Nov 14 14:06:19 2018 -0500

    Merge pull request #791 from michaelkro/773-conversion-host-name

    [#773] conversion host name

    (cherry picked from commit 8598e906d308777252bd7b5604f6412d71327cdf)

    https://bugzilla.redhat.com/show_bug.cgi?id=1643558

 app/javascript/data/models/requests.js | 3 +-
 app/javascript/react/screens/App/Overview/overview.requestWithTasks.fixtures.js | 10 -
 app/javascript/react/screens/App/Plan/Plan.js | 13 +-
 app/javascript/react/screens/App/Plan/PlanActions.js | 36 +-
 app/javascript/react/screens/App/Plan/PlanConstants.js | 4 +
 app/javascript/react/screens/App/Plan/PlanReducer.js | 31 +-
 app/javascript/react/screens/App/Plan/components/PlanRequestDetailList/PlanRequestDetailList.js | 20 +-
 app/javascript/react/screens/App/Plan/helpers.js | 1 -
 app/javascript/react/screens/App/Plan/plan.planRequests.fixtures.js | 4 -
 app/javascript/react/screens/App/common/ListViewToolbar/sortFilter.js | 1 -
 10 files changed, 95 insertions(+), 28 deletions(-)

Comment 9 Yadnyawalk Tale 2018-11-30 09:32:26 UTC
Created attachment 1510091 [details]
popover.png

Though we don't have migration option in UI for beta, I can still do migration with direct URL. Checked with OSP migration plan, I am able to see (earlier-like conversion host instance name on pop-over.

cfme - 5.10.0.27.20181128170555_43ed8cb
v2v plugin - 5.10.z@430ca5fe