Don't queue metrics capture if metrics unsupported

If metrics capture is unsupported by the provider then do not queue perf_capture for targets on that EMS. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1610449
ManageIQ · Aug 8, 2018 · 377d868 · 377d868
1 parent 8a79376
commit 377d868
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/app/models/ext_management_system.rb b/app/models/ext_management_system.rb
@@ -292,6 +292,10 @@ def supports_authentication?(authtype)
     authtype.to_s == "default"
   end
 
+  def supports_metrics?
+    true
+  end
+
   # UI method for determining which icon to show for a particular EMS
   def image_name
     emstype.downcase

diff --git a/app/models/metric/targets.rb b/app/models/metric/targets.rb
@@ -55,6 +55,8 @@ def self.capture_container_targets(emses, _options)
 
     targets = []
     emses.each do |ems|
+      next unless ems.supports_metrics?
+
       targets += with_archived(ems.all_container_nodes)
       targets += with_archived(ems.all_container_groups)
       targets += with_archived(ems.all_containers)