Bug 1233277

Summary: widget generation issues with groups that have no userid set
Product: Red Hat CloudForms Management Engine Reporter: Lucy Fu <lufu>
Component: InsightAssignee: Lucy Fu <lufu>
Status: CLOSED ERRATA QA Contact: Nandini Chandra <nachandr>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.3.0CC: dajohnso, dclarizi, fdewaley, gblomqui, gmccullo, lufu, mfeifer, nachandr, obarenbo, pkovar, thenness
Target Milestone: GAKeywords: ZStream
Target Release: 5.3.5   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.3.5.1 Doc Type: Bug Fix
Doc Text:
In the previous version of CloudForms Management Engine, when the user that created a widget was deleted, the associated dashboard was not properly removed under certain circumstances. As a result, some widgets could not be generated and reports were generated with errors. This problem has been fixed by skipping the deleted user so that widgets are now generated properly.
Story Points: ---
Clone Of: 1227426 Environment:
Last Closed: 2015-07-15 12:55:50 UTC Type: Bug
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: 1227426, 1233412    
Bug Blocks:    

Comment 2 Thomas Hennessy 2015-06-18 15:24:02 UTC
Lucy,
If the existing widgets or their schedules are modified, will that correct the problem as it exists in the customer database, since assumedly those users showing up as 'nil' would no longer be associated with the group or represented in the database.  Please advise.

Comment 4 CFME Bot 2015-06-18 17:51:11 UTC
New commit detected on cfme/5.3.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3a709a39cc410b3d6b263b1fbf0148dbe483e58f

commit 3a709a39cc410b3d6b263b1fbf0148dbe483e58f
Merge: 3547a15 7bce388
Author:     Greg McCullough <gmccullo>
AuthorDate: Thu Jun 18 13:45:19 2015 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Thu Jun 18 13:45:19 2015 -0400

    Merge branch 'widget_nil_user_1233277_53' into '5.3.z'
    
    Fix MiqWidget#grouped_subscribers
    
    to skip the user that does not exist any more.
    
    It is a clean cherry pick for 5.3.z.
    The upstream PR is [here](https://github.com/ManageIQ/manageiq/pull/3115).
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1233277
    
    See merge request !134

 vmdb/app/models/miq_widget.rb       | 13 ++++---
 vmdb/spec/models/miq_widget_spec.rb | 72 +++++++++++++++++++++++--------------
 2 files changed, 52 insertions(+), 33 deletions(-)

Comment 5 CFME Bot 2015-06-18 17:54:28 UTC
New commit detected on cfme/5.3.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3a709a39cc410b3d6b263b1fbf0148dbe483e58f

commit 3a709a39cc410b3d6b263b1fbf0148dbe483e58f
Merge: 3547a15 7bce388
Author:     Greg McCullough <gmccullo>
AuthorDate: Thu Jun 18 13:45:19 2015 -0400
Commit:     Greg McCullough <gmccullo>
CommitDate: Thu Jun 18 13:45:19 2015 -0400

    Merge branch 'widget_nil_user_1233277_53' into '5.3.z'
    
    Fix MiqWidget#grouped_subscribers
    
    to skip the user that does not exist any more.
    
    It is a clean cherry pick for 5.3.z.
    The upstream PR is [here](https://github.com/ManageIQ/manageiq/pull/3115).
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1233277
    
    See merge request !134

 vmdb/app/models/miq_widget.rb       | 13 ++++---
 vmdb/spec/models/miq_widget_spec.rb | 72 +++++++++++++++++++++++--------------
 2 files changed, 52 insertions(+), 33 deletions(-)

Comment 8 Felix Dewaleyne 2015-06-30 14:10:12 UTC
is this fix included in 3.2 as of now? Simon is looking into upgrading to 3.2

Comment 17 Nandini Chandra 2015-07-07 17:49:43 UTC
Verified in 5.3.5

Verification Steps:

1)User - widget, Group - EvmGroup-super_administrator
2)Logged in as user widget,created 'Vendor and guest OS-widget' dashboard widget;
The widget is visible to all users.
3)Deleted 'widget' user manually from DB
5)Logged in as 'admin' and 'Vendor and guest OS-widget' continues to get scheduled

Comment 19 errata-xmlrpc 2015-07-15 12:55:50 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/RHBA-2015-1223.html