Bug 1643558 - [v2v] No conversion host hostname on details page popover
Summary: [v2v] No conversion host hostname on details page popover
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: V2V
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: GA
: 5.10.0
Assignee: Brett Thurber
QA Contact: Yadnyawalk Tale
Red Hat CloudForms Documentation
URL:
Whiteboard: v2v
Depends On: 1643938
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-10-26 14:25 UTC by Yadnyawalk Tale
Modified: 2019-02-12 16:50 UTC (History)
8 users (show)

Fixed In Version: 5.10.0.25
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-12 16:50:45 UTC
Category: Bug
Cloudforms Team: V2V
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
conversation_host_popover.png (59.54 KB, image/png)
2018-10-26 14:25 UTC, Yadnyawalk Tale
no flags Details
popover.png (60.52 KB, image/png)
2018-11-30 09:32 UTC, Yadnyawalk Tale
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github ManageIQ manageiq-v2v issues 773 0 None None None 2018-11-06 10:48:32 UTC

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


Note You need to log in before you can comment on or make changes to this bug.