Bug 1349876

Summary: SSUI : Blank virtual machine row is displayed for service with no VM
Product: Red Hat CloudForms Management Engine Reporter: John Prause <jprause>
Component: UI - OPSAssignee: Shveta <sshveta>
Status: CLOSED ERRATA QA Contact: Shveta <sshveta>
Severity: low Docs Contact:
Priority: medium    
Version: 5.6.0CC: cpelland, dclarizi, hkataria, jhardy, mpovolny, obarenbo
Target Milestone: GAKeywords: ZStream
Target Release: 5.6.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ssui
Fixed In Version: 5.6.1.2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1349601 Environment:
Last Closed: 2016-08-18 17:51:52 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1349601    
Bug Blocks:    
Attachments:
Description Flags
no vm none

Comment 3 CFME Bot 2016-07-11 22:01:37 UTC
New commit detected on ManageIQ/manageiq-ui-self_service/master:
https://github.com/ManageIQ/manageiq-ui-self_service/commit/3192cd3f32cafd1403a8daa746bd8ab0aceaa1e0

commit 3192cd3f32cafd1403a8daa746bd8ab0aceaa1e0
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Jun 24 09:48:21 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Jul 11 17:22:25 2016 -0400

    Initialized vms array for cases when vms was returned as undefined.
    
    - When a service didnt have vms linked to it, vms was coming back as undefined, causing JS errors and not displaying proper message on screen.
    - Added "Power State" label in VM details section when power state iset unknown or set to never, other states show font icon instead of label text.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1349601
    https://bugzilla.redhat.com/show_bug.cgi?id=1349876

 client/app/states/services/details/details.html     | 8 ++++----
 client/app/states/services/details/details.state.js | 6 ++++++
 2 files changed, 10 insertions(+), 4 deletions(-)

Comment 4 CFME Bot 2016-07-11 22:01:45 UTC
New commit detected on ManageIQ/manageiq-ui-self_service/master:
https://github.com/ManageIQ/manageiq-ui-self_service/commit/c35d73ff5f04a24d6eed99fb8184a7bb14834e0e

commit c35d73ff5f04a24d6eed99fb8184a7bb14834e0e
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Jul 1 14:20:43 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Jul 11 17:37:53 2016 -0400

    Fixed div class name.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1349601
    https://bugzilla.redhat.com/show_bug.cgi?id=1349876

 client/app/states/services/details/details.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment 5 CFME Bot 2016-07-11 22:01:53 UTC
New commit detected on ManageIQ/manageiq-ui-self_service/master:
https://github.com/ManageIQ/manageiq-ui-self_service/commit/1e600a4a98af885e927f1379cda9264156424a64

commit 1e600a4a98af885e927f1379cda9264156424a64
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Jun 24 10:48:54 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Jul 11 17:22:46 2016 -0400

    Addressed gulp test failure.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1349601
    https://bugzilla.redhat.com/show_bug.cgi?id=1349876

 .../app/states/services/details/details.state.js   | 39 ++++++++++++----------
 1 file changed, 21 insertions(+), 18 deletions(-)

Comment 6 Harpreet Kataria 2016-07-12 19:24:24 UTC
waiting for https://github.com/ManageIQ/manageiq-ui-self_service/pull/67 to be back-ported

Comment 8 Shveta 2016-08-01 23:34:57 UTC
Created attachment 1186570 [details]
no vm

Fixed.
Verified in 5.6.1.0.20160726144018_810a94f

Comment 10 errata-xmlrpc 2016-08-18 17:51:52 UTC
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://rhn.redhat.com/errata/RHSA-2016-1634.html