Merge pull request #115 from douglasgabriel/support_indentify_leds

Supporting differents identify leds name of physical servers on refresh
ManageIQ · Nov 21, 2017 · c0b7889 · c0b7889
2 parents d6d5117 + 64fd5c8
commit c0b7889
Showing 1 changed file with 3 additions and 10 deletions.
diff --git a/app/models/manageiq/providers/lenovo/physical_infra_manager/refresh_parser.rb b/app/models/manageiq/providers/lenovo/physical_infra_manager/refresh_parser.rb
@@ -372,16 +372,9 @@ def parse_asset_details(node)
     end
 
     def find_loc_led_state(leds)
-      loc_led_state = ""
-      unless leds.nil?
-        leds.each do |led|
-          if led["name"] == "Identify"
-            loc_led_state = led["state"]
-            break
-          end
-        end
-      end
-      loc_led_state
+      identification_led_names = %w(Identify Identification)
+      identification_led = leds.to_a.find { |led| identification_led_names.include?(led["name"]) }
+      identification_led.try(:[], "state")
     end
 
     def discover_ip_physical_infra