From 30a00525056f3fbf456a1c32c9109b9d64093f28 Mon Sep 17 00:00:00 2001 From: Adam Grare Date: Wed, 8 Nov 2017 08:37:36 -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 --- .../openshift/container_manager/inventory_collector_worker.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/models/manageiq/providers/openshift/container_manager/inventory_collector_worker.rb b/app/models/manageiq/providers/openshift/container_manager/inventory_collector_worker.rb index cd1ed50c..8b88cbcc 100644 --- a/app/models/manageiq/providers/openshift/container_manager/inventory_collector_worker.rb +++ b/app/models/manageiq/providers/openshift/container_manager/inventory_collector_worker.rb @@ -2,6 +2,7 @@ class ManageIQ::Providers::Openshift::ContainerManager::InventoryCollectorWorker 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