Project

General

Profile

Actions
Copy link

Bug #16238

closed

dashboard widget for latests events can be exteramlly slow

Added by Ohad Levy almost 9 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ohad Levy
Category:
Performance
Target version:
1.12.3
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/3764
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

when having many hosts, the latest event widget can take around a minute to load.

the line in question is most likely the issue:

 def latest_events
    # 9 reports + header fits the events box nicely...
    @latest_events ||= ConfigReport.authorized(:view_config_reports).my_reports.interesting.where(:host_id => hosts.pluck(:id)).search_for('reported > "7 days ago"').limit(9).includes(:host)
 end

Related issues 4 (0 open — 4 closed)

Related to Foreman - Bug #16044: Errors in individual dashboard widgets fail entire pageClosedTomer Brisker08/10/2016Actions
Related to Foreman - Bug #16627: Puppet general tab new error on 1.12.3ClosedTomer Brisker09/20/2016Actions
Related to Foreman - Bug #16704: Searching fails on dashboardClosedTomer Brisker09/27/2016Actions
Related to Foreman - Bug #18893: dashboard widget for puppet report latest events can be very slow on large amount of reportsClosedTomer Brisker03/14/2017Actions
Actions
Copy link
 #1

Updated by Ohad Levy almost 9 years ago

just changing the query to inner query, saved about 70% of the load time, yet its far from being optimal.

    @latest_events ||= ConfigReport.authorized(:view_config_reports).my_reports.interesting.where(:host => hosts.reorder('')).search_for('reported > "7 days ago"').limit(9).includes(:host)
Actions
Copy link
 #2

Updated by Ohad Levy almost 9 years ago

  • Related to Bug #16044: Errors in individual dashboard widgets fail entire page added
Actions
Copy link
 #3

Updated by The Foreman Bot almost 9 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ohad Levy
  • Pull request https://github.com/theforeman/foreman/pull/3764 added
Actions
Copy link
 #4

Updated by Ohad Levy almost 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link
 #5

Updated by Dominic Cleal almost 9 years ago

  • Translation missing: en.field_release set to 181
Actions
Copy link
 #6

Updated by Dominic Cleal over 8 years ago

  • Related to Bug #16627: Puppet general tab new error on 1.12.3 added
Actions
Copy link
 #7

Updated by Dominic Cleal over 8 years ago

  • Related to Bug #16704: Searching fails on dashboard added
Actions
Copy link
 #8

Updated by Dominic Cleal over 8 years ago

Release set to 1.12.3

Reverted again in 1.12.4 due to too many regressions, and the severe impact of errors on the dashboard. The change remains in 1.13.0 at this time as the errors (e.g. #16704) are more limited.

Actions
Copy link
 #9

Updated by Ohad Levy over 8 years ago

  • Related to Bug #18893: dashboard widget for puppet report latest events can be very slow on large amount of reports added
Actions
Copy link

Also available in: Atom PDF