Bug 1402786

Summary: policy events appear w/o information which entity those belong to in Timelines
Product: Red Hat CloudForms Management Engine Reporter: Ievgen Zapolskyi <izapolsk>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Ievgen Zapolskyi <izapolsk>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.7.0CC: hkataria, jhardy, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:timelines:usability
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1411881 (view as bug list) Environment:
Last Closed: 2017-06-12 17:16:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1411881    
Attachments:
Description Flags
screenshot none

Description Ievgen Zapolskyi 2016-12-08 10:53:02 UTC
Created attachment 1229437 [details]
screenshot

Description of problem:
If there is a vm with some policy events and I open timelines with policy events for provider/host/cluster, I see policy events w/o any information which entity those events belong to. See screenshot.

Actually, I'm not sure that policy events belonging to one entity should be displayed in timelines for another entity.
But if such events should be displayed, those should have appropriate information about owner like it is done for management events. 

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

How reproducible:
100%

Steps to Reproduce:
1. Create policy with condition and events which will produce policy events when vm's PowerState is changed (f.e Vm's PowerState is not Null)
2. Create a policy profile and add created above policy to it
3. Assign the policy profile to some vm
4. Start/Stop that vm
5. Make sure that appropriate policy events appeared in policy_events table of appliance's db
6. Open Timelines for f.e provider this vm belongs to
7. Setup Timelines to show the policy events of all categories and click Apply
8. Click appeared events and look at their details

Actual results:
policy events in provider's timelines appear w/o any information which entity those events belong to

Expected results:
If the policy events of underneath entity like vm, should appear in timelines for provider and so on. Then policy events should have enough information about their possessor.

If the policy events of one entity shouldn't appear in timelines for another entity. Then there shouldn't be any vms policy events in timelines for provider and so on.

Comment 2 Ievgen Zapolskyi 2016-12-08 16:10:38 UTC
this is true for Cloud Intel->Timelines->Policy Events ... Sample 1 report.

Comment 3 Harpreet Kataria 2016-12-13 22:28:28 UTC
https://github.com/ManageIQ/manageiq/pull/13162

Comment 4 CFME Bot 2016-12-16 22:01:26 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/c88ab80ebedda22777bf7f3ebd144bb464dda81c

commit c88ab80ebedda22777bf7f3ebd144bb464dda81c
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue Dec 13 17:21:19 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Dec 16 14:02:18 2016 -0500

    Added missing target information and links on Policy Event bubbles.
    
    changes in 4f650a999c893b7519d9e7648acea92b027f59ff introduced the issue variable e_text was initialized to '' ignoring the fact that it was being set in several places above in the code. Fixed code to use TimelineMessage class to set links and other target related information for PolicyEvent timelines event bubble.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1402786

 lib/report_formatter/timeline.rb                   | 36 ----------------------
 lib/report_formatter/timeline_message.rb           | 26 ++++++++++++++++
 .../miq_reports/tl_policy_events_daily.yaml        |  7 ++++-
 .../miq_reports/tl_policy_events_hourly.yaml       |  5 +++
 spec/lib/report_formater/timeline_spec.rb          | 23 ++++++++++++++
 5 files changed, 60 insertions(+), 37 deletions(-)