Plugins » Discovery

Discovery rules are processed in order of :name, not :priority.

A simple fix is in line 9 of foreman_discovery-4.0.0/app/controllers/concerns/foreman/controller/discovered_extensions.rb:

-    DiscoveryRule.where(:enabled => true).reorder(:priority).each do |rule|
+    DiscoveryRule.where(:enabled => true).order(:priority).each do |rule|

I don't know for sure, but think it could be related to foreman_discovery-4.0.0/app/models/discovery_rule.rb, which sets: order("discover_rules.name") around line 29:

  # with proc support, default_scope can no longer be chained
  # include all default scoping here
  default_scope lambda {
                  with_taxonomy_scope do
                    order("discovery_rules.name")
                  end
                }