Omit rack id field when storing physical server

Field in question should point to an existing physical rack. At the moment, Redfish provider does not support adding racks to the inventory, so this field should be omitted when saving the server.
ManageIQ · Jun 14, 2018 · 2855182 · 2855182
1 parent fce4788
commit 2855182
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/app/models/manageiq/providers/redfish/inventory/parser/physical_infra_manager.rb b/app/models/manageiq/providers/redfish/inventory/parser/physical_infra_manager.rb
@@ -24,8 +24,7 @@ def physical_servers
           :field_replaceable_unit => "dummy",
           :raw_power_state        => s["PowerState"],
           :vendor                 => "unknown",
-          :location_led_state     => s["IndicatorLED"],
-          :physical_rack_id       => 0
+          :location_led_state     => s["IndicatorLED"]
         )
         persister.computer_systems.build(:managed_entity => server)
       end

diff --git a/spec/models/manageiq/providers/redfish/physical_infra_manager/refresher_spec.rb b/spec/models/manageiq/providers/redfish/physical_infra_manager/refresher_spec.rb
@@ -44,8 +44,7 @@ def assert_physical_servers
       :field_replaceable_unit => "dummy",
       :raw_power_state        => "Off",
       :vendor                 => "unknown",
-      :location_led_state     => "Off",
-      :physical_rack_id       => 0
+      :location_led_state     => "Off"
     )
   end