From 20a984a1b40eaef48afdb2fd93e12f5220061701 Mon Sep 17 00:00:00 2001 From: Adam Grare Date: Wed, 20 Apr 2022 12:36:34 -0400 Subject: [PATCH] Use auto-model-class for ContainerImages Having the `:model_class` set to `::ContainerImage` meant that auto_model_class was not being called, causing any records which did not have a `:type` explicitly set by the parser would end up having a type set to the base `::ContainerImage` class. --- .../providers/inventory/persister/builder/container_manager.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/app/models/manageiq/providers/inventory/persister/builder/container_manager.rb b/app/models/manageiq/providers/inventory/persister/builder/container_manager.rb index 9c687c807802..f89b6962636c 100644 --- a/app/models/manageiq/providers/inventory/persister/builder/container_manager.rb +++ b/app/models/manageiq/providers/inventory/persister/builder/container_manager.rb @@ -89,7 +89,6 @@ def container_images # TODO: (bpaskinc) old save matches on [:image_ref, :container_image_registry_id] # TODO: (bpaskinc) should match on digest when available # TODO: (mslemr) provider-specific class exists (openshift), but specs fail with them (?) - :model_class => ::ContainerImage, :manager_ref => %i[image_ref], :delete_method => :disconnect_inv, :custom_reconnect_block => custom_reconnect_block