It looks like this query is very expensive on MySQL: host.fact_values.where(:fact_name => fact_names.values).reorder('')

We call it twice currently while it is not needed to run this query twice, we can fetch the list of fact value IDs stored in database in update step and prepare list of facts to be added in the next step in advance.

Also we order the query by ID for some reason, we can process them unordered just fine.

https://community.theforeman.org/t/foreman-1-18-2-fact-query-on-puppet-report-upload-fact-import-incredibly-slow/11238