Project

General

Profile

Actions

Bug #16238

closed

dashboard widget for latests events can be exteramlly slow

Added by Ohad Levy over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Performance
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

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 open4 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 #1

Updated by Ohad Levy over 7 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 #2

Updated by Ohad Levy over 7 years ago

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

Updated by The Foreman Bot over 7 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 #4

Updated by Ohad Levy over 7 years ago

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

Updated by Dominic Cleal over 7 years ago

  • translation missing: en.field_release set to 181
Actions #6

Updated by Dominic Cleal over 7 years ago

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

Updated by Dominic Cleal over 7 years ago

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

Updated by Dominic Cleal over 7 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 #9

Updated by Ohad Levy about 7 years ago

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

Also available in: Atom PDF