Actions
Bug #13112
closedN+1 query on hosts#index from last report object
Description
Since #12954, the hosts index is reporting an N+1 database lookup when retrieving the last report object to calculate the host status.
There's an optimisation in #10782 which isn't being used when calling the relevant? method (which now uses the object) as the method doesn't take options including the last report object.
Updated by Dominic Cleal about 9 years ago
- Related to Bug #12954: Foreman puppet report status does not work without a puppet proxy added
Updated by Dominic Cleal about 9 years ago
- Status changed from New to Assigned
- Assignee set to Dominic Cleal
Updated by Dominic Cleal about 9 years ago
Stack trace:
| N+1 Query method call stack | /home/dcleal/code/foreman/foreman/app/models/host_status/configuration_status.rb:7:in `last_report' | /home/dcleal/code/foreman/foreman/app/models/host_status/configuration_status.rb:83:in `relevant?' | /home/dcleal/code/foreman/foreman/app/models/host_status/global.rb:10:in `block in build' | /home/dcleal/code/foreman/foreman/app/models/host_status/global.rb:10:in `build' | /home/dcleal/code/foreman/foreman/app/models/host/managed.rb:885:in `build_global_status' | /home/dcleal/code/foreman/foreman/app/helpers/hosts_helper.rb:109:in `host_global_status_icon_class_for_host' | /home/dcleal/code/foreman/foreman/app/helpers/hosts_helper.rb:98:in `name_column' | /home/dcleal/code/foreman/foreman/app/views/hosts/_list.html.erb:22:in `block in _app_views_hosts__list_html_erb___2368167352033240693_98631040' | /home/dcleal/code/foreman/foreman/app/views/hosts/_list.html.erb:17:in `_app_views_hosts__list_html_erb___2368167352033240693_98631040' | /home/dcleal/code/foreman/foreman/app/views/hosts/index.html.erb:4:in `_app_views_hosts_index_html_erb__3208504113801726348_98118920'
Updated by The Foreman Bot about 9 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/3055 added
Updated by Dominic Cleal about 9 years ago
- Translation missing: en.field_release changed from 71 to 104
Updated by Dominic Cleal about 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 294ced20fa422e7bb86022d4139f4af91515ef64.
Actions