Project

General

Profile

Actions
Copy link

Bug #5841

closed

Foreman dashboard slow in 1.5 for non admin users

Added by Chuck Schweizer over 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Dominic Cleal
Category:
Performance
Target version:
1.7.1
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/1995
Fixed in Releases:
Found in Releases:
1.5.0
Red Hat JIRA:

Description

Previous ticket that fixed the issue for foreman 1.4
http://projects.theforeman.org/issues/3122
http://projects.theforeman.org/issues/2982

This foreman instance has 25,036 puppet clients reporting to it.

Files

foreman_slow_non_admin_dashboard.txt foreman_slow_non_admin_dashboard.txt 348 KB debug output Chuck Schweizer, 05/20/2014 07:21 PM

Related issues 4 (0 open — 4 closed)

Related to Foreman - Bug #2982: Non Admin user Dashboard load is very slow with a large number of mangaged nodesResolved08/28/2013Actions
Related to Foreman - Bug #3122: when fact filters are added to a user, error column reference "id" is ambiguous for dashboard and fact_valuesClosed09/19/2013Actions
Related to Foreman - Bug #7009: N+1 hosts query in dashboard pageClosedOhad Levy08/10/2014Actions
Related to Foreman - Bug #8817: PGError: ERROR: missing FROM-clause entry for table "hostgroups"ClosedDominic Cleal12/31/2014Actions
Actions
Copy link
 #1

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #2982: Non Admin user Dashboard load is very slow with a large number of mangaged nodes added
Actions
Copy link
 #2

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #3122: when fact filters are added to a user, error column reference "id" is ambiguous for dashboard and fact_values added
Actions
Copy link
 #3

Updated by Dominic Cleal over 10 years ago

  • Difficulty deleted (easy)
Actions
Copy link
 #4

Updated by Ohad Levy over 10 years ago

Dominic, did you investigate on this? Any reason why the ID's are added instead of INNER query?

Actions
Copy link
 #5

Updated by Dominic Cleal over 10 years ago

Ohad Levy wrote:

Dominic, did you investigate on this? Any reason why the ID's are added instead of INNER query?

Not in depth, but I saw here that it was done deliberately: https://github.com/theforeman/foreman/blob/develop/app/models/report.rb#L36-L37

It might be we can use the authorized scope directly, but I haven't tested.

Actions
Copy link
 #6

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #7009: N+1 hosts query in dashboard page added
Actions
Copy link
 #7

Updated by Dominic Cleal about 10 years ago

  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal
Actions
Copy link
 #8

Updated by The Foreman Bot about 10 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1995 added
  • Pull request deleted ()
Actions
Copy link
 #9

Updated by Dominic Cleal about 10 years ago

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

Updated by Dominic Cleal about 10 years ago

  • Translation missing: en.field_release set to 29
Actions
Copy link
 #11

Updated by Dominic Cleal almost 10 years ago

  • Related to Bug #8817: PGError: ERROR: missing FROM-clause entry for table "hostgroups" added
Actions
Copy link
 #12

Updated by Dominic Cleal almost 10 years ago

This was shipped in 1.7.1 but will be reverted in 1.7.3 as it caused a more severe issue recorded in #8817. The change remains in the 1.8/1.9 branches, where I hope to have a better solution to fix both.

Actions
Copy link

Also available in: Atom PDF