From 273e882113de64cc0c41247d2c04bc6b1f1c135d Mon Sep 17 00:00:00 2001 From: Adam Grare Date: Wed, 8 Nov 2017 08:21:48 -0500 Subject: [PATCH] Fix Inventory Collector has_required_role? The has_required_role? method wasn't calling the base MiqWorker#has_required_role? leading to all workers in the zone running the inventory collector. Depends: https://github.com/ManageIQ/manageiq/pull/16415 --- .../kubernetes/container_manager/inventory_collector_worker.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/models/manageiq/providers/kubernetes/container_manager/inventory_collector_worker.rb b/app/models/manageiq/providers/kubernetes/container_manager/inventory_collector_worker.rb index 15e52ebc75..905337cef6 100644 --- a/app/models/manageiq/providers/kubernetes/container_manager/inventory_collector_worker.rb +++ b/app/models/manageiq/providers/kubernetes/container_manager/inventory_collector_worker.rb @@ -2,6 +2,7 @@ class ManageIQ::Providers::Kubernetes::ContainerManager::InventoryCollectorWorke require_nested :Runner def self.has_required_role? - !worker_settings[:disabled] && Settings.fetch_path(:ems_refresh, ems_class.ems_type.to_sym, :inventory_object_refresh) + return false unless Settings.fetch_path(:ems_refresh, ems_class.ems_type.to_sym, :inventory_object_refresh) + super end end