Do not crash if ingress is missing status.loadbalancer.ingress property (#5665)

src/common/k8s-api/__tests__/ingress.api.test.ts

@@ -5,6 +5,53 @@
 
 import { computeRuleDeclarations, Ingress } from "../endpoints";
 
+describe("Ingress", () => {
+  it("given no loadbalancer ingresses in status property, loadbalancers should be an empty array", () => {
+    const ingress = new Ingress({
+      apiVersion: "networking.k8s.io/v1",
+      kind: "Ingress",
+      metadata: {
+        name: "foo",
+        resourceVersion: "1",
+        uid: "bar",
+        namespace: "default",
+        selfLink: "/apis/networking.k8s.io/v1/ingresses/default/foo",
+      },
+      status: {
+        loadBalancer: {},
+      },
+    });
+
+    expect(ingress.getLoadBalancers()).toEqual([]);
+  });
+
+  it("given loadbalancer ingresses in status property, loadbalancers should be flat array of ip adresses and hostnames", () => {
+    const ingress = new Ingress({
+      apiVersion: "networking.k8s.io/v1",
+      kind: "Ingress",
+      metadata: {
+        name: "foo",
+        resourceVersion: "1",
+        uid: "bar",
+        namespace: "default",
+        selfLink: "/apis/networking.k8s.io/v1/ingresses/default/foo",
+      },
+      status: {
+        loadBalancer: {
+          ingress: [{
+            ip: "10.0.0.27",
+          },
+          {
+            hostname: "localhost",
+          }],
+        },
+      },
+    });
+
+    expect(ingress.getLoadBalancers()).toEqual(["10.0.0.27", "localhost"]);
+  });
+});
+
 describe("computeRuleDeclarations", () => {
   it("given no tls field, should format links as http://", () => {
     const ingress = new Ingress({

src/common/k8s-api/endpoints/ingress.api.ts

@@ -136,7 +136,7 @@ export interface IngressSpec {
 
 export interface IngressStatus {
   loadBalancer: {
-    ingress: ILoadBalancerIngress[];
+    ingress?: ILoadBalancerIngress[];
   };
 }
 
@@ -199,7 +199,7 @@ export class Ingress extends KubeObject<IngressStatus, IngressSpec, KubeObjectSc
   }
 
   getLoadBalancers() {
-    return this.status?.loadBalancer.ingress.map(address => (
+    return this.status?.loadBalancer?.ingress?.map(address => (
       address.hostname || address.ip
     )) ?? [];
   }