[wip] display events for physical storage

Author: tonyschneider

app/models/ems_event.rb

@@ -78,6 +78,7 @@ def self.add(ems_id, event_hash)
     process_availability_zone_in_event!(event_hash)
     process_cluster_in_event!(event_hash)
     process_container_entities_in_event!(event_hash)
+    process_physical_storage_in_event!(event_hash)
 
     # Write the event
     new_event = create_event(event_hash)
@@ -136,6 +137,10 @@ def self.process_host_in_event!(event, options = {})
     end
   end
 
+  def self.process_physical_storage_in_event!(event, options = {})
+    process_object_in_event!(PhysicalStorage, event, options)
+  end
+
   def self.process_container_entities_in_event!(event, _options = {})
     [ContainerNode, ContainerGroup, ContainerReplicator].each do |entity|
       process_object_in_event!(entity, event)

app/models/event_stream.rb

@@ -26,6 +26,7 @@ class EventStream < ApplicationRecord
   belongs_to :container_node
 
   belongs_to :physical_server
+  belongs_to :physical_storage
   belongs_to :physical_chassis, :inverse_of => :event_streams
   belongs_to :physical_switch, :inverse_of => :event_streams
 

app/models/physical_storage.rb

@@ -5,34 +5,32 @@ class PhysicalStorage < ApplicationRecord
   include SupportsFeatureMixin
   include CustomActionsMixin
   include EmsRefreshMixin
+  include EventMixin
+
+  # The physical-storage is expected to have san_addresses of its own in the future (The real addresses through which it actually connects to the SAN).
+  # Therefore, the name san_addresses is reserved for the physical-storages actual san-addresses, and for all of the san_addresses configured in the physical-storage's host_initiators we refer as registered_initiator_addresses.
 
   belongs_to :ext_management_system, :foreign_key => :ems_id
   belongs_to :physical_rack
   belongs_to :physical_chassis, :inverse_of => :physical_storages
+  belongs_to :physical_storage_family, :inverse_of => :physical_storages
+
+  has_one :hardware, :through => :computer_system
+  has_one :computer_system, :as => :managed_entity, :dependent => :destroy
+  has_one :asset_detail, :as => :resource, :dependent => :destroy, :inverse_of => false
 
   has_many :storage_resources, :dependent => :destroy
   has_many :host_initiators, :dependent => :destroy
   has_many :host_initiator_groups, :dependent => :destroy
-
-  # The physical-storage is expected to have san_addresses of its own in the future (The real addresses through which it actually connects to the SAN).
-  # Therefore, the name san_addresses is reserved for the physical-storages actual san-addresses, and for all of the san_addresses configured in the physical-storage's host_initiators we refer as registered_initiator_addresses.
   has_many :registered_initiator_addresses, :through => :host_initiators, :source => :san_addresses
-
-  belongs_to :physical_storage_family, :inverse_of => :physical_storages
-
-  has_one :asset_detail, :as => :resource, :dependent => :destroy, :inverse_of => false
-
   has_many :canisters, :dependent => :destroy, :inverse_of => false
   has_many :physical_disks, :dependent => :destroy, :inverse_of => :physical_storage
-
-  has_one :computer_system, :as => :managed_entity, :dependent => :destroy
-  has_one :hardware, :through => :computer_system
-
   has_many :canister_computer_systems, :through => :canisters, :source => :computer_system
   has_many :guest_devices, :through => :hardware
-
   has_many :wwpn_candidates, :dependent => :destroy
 
+  supports :timeline
+
   supports_not :create
   supports_not :delete
   supports_not :update_physical_storage
@@ -163,4 +161,8 @@ def update_physical_storage(options = {})
   def raw_update_physical_storage(_options = {})
     raise NotImplementedError, _("raw_update_volume must be implemented in a subclass")
   end
+
+  def event_where_clause(assoc = :ems_events)
+    ["#{events_table_name(assoc)}.physical_storage_id = ?", id]
+  end
 end

db/fixtures/miq_product_features.yml

@@ -6645,6 +6645,10 @@
       :description: Display Individual Physical Storage
       :feature_type: view
       :identifier: physical_storage_show
+    - :name: Timeline
+      :description: Display Timelines for Physical Storage
+      :feature_type: view
+      :identifier: physical_storage_timeline
   - :name: Operate
     :description: Perform Operations on Physical Storages
     :feature_type: control
@@ -6679,7 +6683,6 @@
       :description: Validate a Physical Storage
       :feature_type: admin
       :identifier: physical_storage_validate
-
 # Physical Servers
 - :name: Physical Servers
   :description: Everything under Physical Servers