Merge pull request #82 from avivtur/116-manual-backport

OCPBUGS-49650: Manual backport for #116

Author: openshift-merge-bot[bot]

src/views/states/topology/utils/utils.ts

@@ -62,10 +62,10 @@ const createNodes = (
 const createEdges = (childNodes: NodeModel[]): EdgeModel[] => {
   const edges: EdgeModel[] = [];
 
-  childNodes.forEach((sourceNode) => {
+  childNodes?.forEach((sourceNode) => {
     // Find bridge connections
     if (!isEmpty(sourceNode.data?.bridgePorts)) {
-      sourceNode.data?.bridgePorts.forEach((port) => {
+      sourceNode.data?.bridgePorts?.forEach((port) => {
         const targetNode = childNodes.find(
           (target) => target.label === port.name && target.id !== sourceNode.id,
         );
@@ -82,8 +82,8 @@ const createEdges = (childNodes: NodeModel[]): EdgeModel[] => {
     }
 
     // Find bond connections
-    if (!isEmpty(sourceNode.data?.vlanBaseInterface)) {
-      sourceNode.data?.bondPorts.forEach((port) => {
+    if (!isEmpty(sourceNode.data?.bondPorts)) {
+      sourceNode.data?.bondPorts?.forEach((port) => {
         const targetNode = childNodes.find(
           (target) => target.label === port && target.id !== sourceNode.id,
         );
@@ -100,7 +100,7 @@ const createEdges = (childNodes: NodeModel[]): EdgeModel[] => {
     }
 
     // Find vlan connections
-    if (!isEmpty(sourceNode.data?.bondPorts)) {
+    if (!isEmpty(sourceNode.data?.vlanBaseInterface)) {
       const baseInterface = sourceNode.data?.vlanBaseInterface;
 
       const targetNode = childNodes.find(
@@ -145,7 +145,7 @@ export const transformDataToTopologyModel = (
   const nodes: NodeModel[] = [];
   const edges: EdgeModel[] = [];
 
-  data.forEach((nodeState) => {
+  data?.forEach((nodeState) => {
     const nnsName = nodeState.metadata.name;
 
     const childNodes = createNodes(nnsName, nodeState.status.currentState.interfaces);