Adds role check before queuing the job;

Also removes the check for server zone

Author: elsamaryv

app/controllers/mixins/ems_common/metrics.rb

@@ -72,7 +72,13 @@ def refresh_or_capture_emss(task, action_name)
         if task == "refresh_ems"
           model.refresh_ems(emss, true)
         elsif task == "capture_ems"
-          emss.each(&:queue_metrics_capture)
+          emss.each do |ems|
+            unless ems.zone.role_active?("ems_metrics_coordinator")
+              add_flash(_("Capacity & Utilization Coordinator role is off for %{name}") % {:name => ems.name}, :error)
+            else
+              ems.queue_metrics_capture
+            end
+          end
         end
 
         add_task_flash(emss, action_name)

app/helpers/application_helper/button/ems_capture_metrics.rb

@@ -5,9 +5,8 @@ def disabled?
       check_credentials
       check_refresh
       check_endpoint
+      check_role
     end
-    check_role
-
     @error_message.present?
   end
 
@@ -26,7 +25,7 @@ def check_endpoint
   def check_role
     metrics_collection =
       ManageIQ::Providers::Kubernetes::ContainerManager.method_defined?(:queue_metrics_capture) &&
-        (@record ? @record.zone.role_active?("ems_metrics_coordinator") : MiqServer.my_server.zone.role_active?("ems_metrics_coordinator"))
+        @record.zone.role_active?("ems_metrics_coordinator")
 
     @error_message ||= _("Capacity & Utilization Coordinator role is off") unless metrics_collection
   end