Bug 1526058

Summary: custom reports not visible to group/role that could see them prior to recent upgrade
Product: Red Hat CloudForms Management Engine Reporter: Saif Ali <saali>
Component: ReportingAssignee: Gregg Tanzillo <gtanzill>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: high Docs Contact:
Priority: high    
Version: 5.8.0CC: obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly
Target Release: 5.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1531301 1532857 (view as bug list) Environment:
Last Closed: 2018-06-21 20:48:08 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:    
Bug Blocks: 1531301, 1532857    

Description Saif Ali 2017-12-14 17:13:10 UTC
Description of problem:
custom reports not visible to group/role that could see them prior to recent upgrade.  the group cfme-tagger used to be able to run some custom reports in v4.0.  Now those reports are only visible to EVM_superadministrator


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

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 5 CFME Bot 2018-01-04 22:21:00 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/597804725b87a842c5c83bd5e26536e3d45b2716

commit 597804725b87a842c5c83bd5e26536e3d45b2716
Author:     Gregg Tanzillo <gtanzill>
AuthorDate: Thu Dec 21 16:27:28 2017 -0500
Commit:     Gregg Tanzillo <gtanzill>
CommitDate: Thu Jan 4 16:04:59 2018 -0500

    Expand scope of report definitions that a user can see
    
    This change expands the existing scope of the current user's group to the current user's tenant
    allowing visibility to all reports created by users within the tenant.
    
    The scope was changed in https://github.com/ManageIQ/manageiq/pull/15472 but was too restrictive
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1526058

 app/models/miq_report.rb       |  2 +-
 spec/models/miq_report_spec.rb | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)