Actions
Bug #16576
closed
Structured facts importer is extremely slow
Description
After upgrading to 1.13.0-RC1, our Foreman server consumed all available resources and became effectively unusable with response times of 25k to 175k ms. This seems directly related to the changes made in #4528, as Api::V2::HostsController#facts is where the application spends 99% of its time after upgrade according to NewRelic.
Updated by 
Updated by 
Updated by
Added by Dominic Cleal over 8 years ago
Added by Dominic Cleal over 8 years ago
fixes #16576 - only load FactName IDs, not models during import
Restores behaviour from before c696f74, which only keeps FactName IDs to
avoid loading the entire model. For structured facts, the compose values
are also preloaded, so the model is only loaded and updated if the value
of compose changes from the preloaded value.
(cherry picked from commit e5381d3ff9001d9fe3ce8d0f90afeec591571e97)
Actions
fixes #16576 - only load FactName IDs, not models during import
Restores behaviour from before c696f74, which only keeps FactName IDs to
avoid loading the entire model. For structured facts, the compose values
are also preloaded, so the model is only loaded and updated if the value
of compose changes from the preloaded value.