Project

General

Profile

Bug #8416

Puppetclasses edit n+1 query

Added by Daniel Lobato Garcia over 4 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Category:
Database
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Lookup_values and environment_classes should be eager loaded on edits for Puppetclasses to avoid two slow n+1 queries.


Related issues

Related to Foreman - Bug #9981: Smart variable doesn't show error on new variableClosed2015-04-01

Associated revisions

Revision c4f9aca6 (diff)
Added by Daniel Lobato Garcia over 4 years ago

Fixes #8416 - Fix Puppet classes form n+1 queries

Revision 6db663cd (diff)
Added by Daniel Lobato Garcia over 4 years ago

Fixes #8416 - Fix Puppet classes form n+1 queries

(cherry picked from commit c4f9aca6da11d2e37446fd0f60bcf515358e726e)

History

#1 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1945 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal over 4 years ago

  • Category set to Database
  • Assignee set to Daniel Lobato Garcia
  • Target version set to 1.7.2

#3 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 29

#4 Updated by Daniel Lobato Garcia over 4 years ago

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

#5 Updated by Dominic Cleal almost 4 years ago

  • Related to Bug #9981: Smart variable doesn't show error on new variable added

Also available in: Atom PDF