Foreman » Installer

Dominic Cleal wrote:

The script already has a :timeout setting, which is set to read_timeout (and open_timeout in the latest version) when performing fact uploads. Shouldn't that change the same thing?

Sort of. I don't claim to have advanced knowledge of the foreman code base, or even Ruby, but it seems that :timeout for node.rb controls a timeout for the block of code that gets the classification of the server, not the read_timeout for the Net::HTTP client that pushes the facts into the foreman API.

I think I found the cause of the slowness, it seems like it's ActiveRecord:

app/models/host/base.rb:

      if iface.new_record? || iface.changed?
        logger.debug "Saving #{name} NIC for host #{self.name}" 
        result = iface.save

        unless result
          logger.warn "Saving #{name} NIC for host #{self.name} failed, skipping because:" 
          iface.errors.full_messages.each { |e| logger.warn " #{e}" }
        end

        result
      end

If I disable activerecord callbacks (iface.save to iface.save(:validate => false)) this thing goes way way faster. I'm talking like, we're going from 5 minutes to import these facts, to 14 seconds.