Stop nils from appearing in list of hosts to test OSP ssh keys against

ManageIQ · Jan 10, 2017 · 3c80a31 · 3c80a31
1 parent 95e002b
commit 3c80a31
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/app/models/manageiq/providers/openstack/infra_manager.rb b/app/models/manageiq/providers/openstack/infra_manager.rb
@@ -115,9 +115,11 @@ def required_credential_fields(type)
   end
 
   def verify_ssh_keypair_credentials(_options)
+    # Select one powered-on host in each cluster to verify
+    # ssh credentials against
     hosts.sort_by(&:ems_cluster_id)
          .slice_when { |i, j| i.ems_cluster_id != j.ems_cluster_id }
-         .map { |c| c.find { |h| h.power_state == 'on' } }
+         .map { |c| c.find { |h| h.power_state == 'on' } }.compact
          .all? { |h| h.verify_credentials('ssh_keypair') }
   end
   private :verify_ssh_keypair_credentials