Project

General

Profile

Actions

Bug #2498

closed

Reports yaml import is very slow

Added by Daniel Lobato Garcia over 11 years ago. Updated over 11 years ago.

Status:
Closed
Priority:
High
Category:
Importers
Target version:
Difficulty:
easy
Triaged:
Fixed in Releases:
Found in Releases:

Description

facts.keys.dup.delete_if { |n| fact_names.map(&:name).include? n } creates a tremendous overhead on the facts import from the puppet masters, this fix simply takes the code from the original Puppet::Rails::Host. As an example, imports have gone from ~30s to ~200ms

https://github.com/theforeman/foreman/pull/615


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #2509: fact names are duplicated in the databaseClosed05/20/2013Actions
Related to Foreman - Bug #2507: fact importer failsClosed05/20/2013Actions
Actions #1

Updated by Greg Sutcliffe over 11 years ago

  • Status changed from New to Ready For Testing
Actions #2

Updated by Daniel Lobato Garcia over 11 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF