Merge pull request #943 from yaacov/storage-map-details-page

🐾 Storage map details page

packages/forklift-console-plugin/locales/en/plugin__forklift-console-plugin.json

@@ -99,6 +99,7 @@
   "Delete NetworkMap?": "Delete NetworkMap?",
   "Delete Plan?": "Delete Plan?",
   "Delete Provider": "Delete Provider",
+  "Delete StorageMap": "Delete StorageMap",
   "Delete StorageMap?": "Delete StorageMap?",
   "Description": "Description",
   "Details": "Details",
@@ -204,7 +205,6 @@
   "Migration networks maps are used to map network interfaces between source and target workloads.": "Migration networks maps are used to map network interfaces between source and target workloads.",
   "Migration plans are used to plan migration or virtualization workloads from source providers to target providers.": "Migration plans are used to plan migration or virtualization workloads from source providers to target providers.",
   "Migration started": "Migration started",
-  "Migration storage maps are used to map storage interfaces between source and target workloads.": "Migration storage maps are used to map storage interfaces between source and target workloads.",
   "Migration Toolkit for Virtualization": "Migration Toolkit for Virtualization",
   "Migrations": "Migrations",
   "Migrations (last 24 hours)": "Migrations (last 24 hours)",
@@ -375,6 +375,8 @@
   "Storage Map name re-generated": "Storage Map name re-generated",
   "Storage map:": "Storage map:",
   "Storage mappings have been re-generated": "Storage mappings have been re-generated",
+  "StorageMap details": "StorageMap details",
+  "StorageMap YAML": "StorageMap YAML",
   "StorageMaps": "StorageMaps",
   "StorageMaps for virtualization": "StorageMaps for virtualization",
   "Storages used by the selected VMs": "Storages used by the selected VMs",
@@ -410,7 +412,6 @@
   "This plan cannot be edited because it is running must gather.": "This plan cannot be edited because it is running must gather.",
   "This plan cannot be edited because the inventory data for its associated providers is not ready.": "This plan cannot be edited because the inventory data for its associated providers is not ready.",
   "This plan cannot be restarted because it is running must gather service": "This plan cannot be restarted because it is running must gather service",
-  "To": "To",
   "To make changes to the plan, select Duplicate and edit the duplicate plan.": "To make changes to the plan, select Duplicate and edit the duplicate plan.",
   "To troubleshoot, check the Forklift controller pod events and logs.": "To troubleshoot, check the Forklift controller pod events and logs.",
   "To troubleshoot, check the Forklift controller pod logs.": "To troubleshoot, check the Forklift controller pod logs.",

packages/forklift-console-plugin/src/modules/NetworkMaps/views/details/components/MapsSection/MapsSection.tsx

@@ -98,6 +98,11 @@ export const MapsSection: React.FC<MapsSectionProps> = ({ obj }) => {
     );
     const nextSourceNet = sourceNetworks.find((n) => n?.name === next.source);
 
+    // sanity check, names may not be valid
+    if (!nextSourceNet) {
+      return;
+    }
+
     const nextMap: V1beta1NetworkMapSpecMap = {
       source: convertInventoryNetworkToV1beta1NetworkMapSpecMapSource(nextSourceNet),
       destination:
@@ -244,4 +249,4 @@ function convertOpenShiftNetworkAttachmentDefinitionToV1beta1NetworkMapSpecMapDe
 }
 
 const OpenShiftNetworkAttachmentDefinitionToName = (net) =>
-  net?.namespace ? `${net?.namespace}/${net?.name}` : net?.name;
+  net?.namespace ? `${net?.namespace}/${net?.name}` : net?.name || 'Pod';