Merge pull request #20772 from kbrock/invalidate_all_caches

Tenant changes changes miq product features on all servers

(cherry picked from commit c7836b2)

app/models/miq_product_feature.rb

@@ -108,6 +108,18 @@ def self.invalidate_caches
     @detail = nil
   end
 
+  # invalidate feature cache on this server and others
+  #
+  # called when data in the features change (typically tenant data).
+  # This then uses the queue to tell other servers they need to update as well.
+  def self.invalidate_caches_queue
+    invalidate_caches
+    MiqQueue.broadcast(
+      :class_name  => name,
+      :method_name => "invalidate_caches"
+    )
+  end
+
   def self.features
     @feature_cache ||= begin
       # create hash with parent identifier and details

app/models/tenant.rb

@@ -63,7 +63,8 @@ class Tenant < ApplicationRecord
   virtual_column :display_type, :type => :string
 
   before_save :nil_blanks
-  after_create :create_tenant_group, :create_miq_product_features_for_tenant_nodes
+  after_save -> { MiqProductFeature.invalidate_caches }
+  after_create :create_tenant_group, :create_miq_product_features_for_tenant_nodes, :update_miq_product_features_for_tenant_nodes
 
   def self.scope_by_tenant?
     true
@@ -319,6 +320,10 @@ def create_miq_product_features_for_tenant_nodes
     MiqProductFeature.seed_single_tenant_miq_product_features(self)
   end
 
+  def update_miq_product_features_for_tenant_nodes
+    MiqProductFeature.invalidate_caches_queue
+  end
+
   def destroy_with_subtree
     subtree.sort_by(&:depth).reverse.each(&:destroy)
   end

spec/models/host_spec.rb

@@ -135,6 +135,7 @@
       it "policy passes" do
         expect_any_instance_of(described_class).to receive(:ipmi_power_on)
 
+        MiqQueue.delete_all
         @host.start
         status, message, result = MiqQueue.first.deliver
         MiqQueue.first.delivered(status, message, result)