From c0d3f44f3553da07dcac7301780bd4ebb459f450 Mon Sep 17 00:00:00 2001 From: Samuel Padgett Date: Sun, 28 Jul 2019 15:43:51 -0400 Subject: [PATCH] Bug 1733806: Avoid runtime errors checking quota when processing template --- app/scripts/services/quota.js | 4 ++-- dist/scripts/scripts.js | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/scripts/services/quota.js b/app/scripts/services/quota.js index 65c1cd6b78..342d94afc7 100644 --- a/app/scripts/services/quota.js +++ b/app/scripts/services/quota.js @@ -71,7 +71,7 @@ angular.module("openshiftConsole") if (resource.kind === 'Pod') { return filterQuotasForPodTemplate(resource, quotas); } - else if (_.has(resource, "spec.template")) { + else if (_.has(resource, "spec.template.spec")) { return filterQuotasForPodTemplate(resource.spec.template, quotas); } // We plan on having other resources that scopes will affect @@ -156,7 +156,7 @@ angular.module("openshiftConsole") var getResourceLimitAlerts = function(resource, quota){ var alerts = []; var podTemplate = resource.kind === "Pod" ? resource : _.get(resource, 'spec.template'); - if (!podTemplate) { + if (!podTemplate || !podTemplate.spec) { // Didn't have a pod template, so we don't care about this resource return alerts; } diff --git a/dist/scripts/scripts.js b/dist/scripts/scripts.js index e29e3c8439..a5faf33886 100644 --- a/dist/scripts/scripts.js +++ b/dist/scripts/scripts.js @@ -3790,7 +3790,7 @@ return !0; }); }); }, S = function(e, t) { -return e ? "Pod" === e.kind ? b(e, t) : _.has(e, "spec.template") ? b(e.spec.template, t) : t : t; +return e ? "Pod" === e.kind ? b(e, t) : _.has(e, "spec.template.spec") ? b(e.spec.template, t) : t : t; }, C = e("humanizeQuotaResource"), w = e("humanizeKind"), P = function(e, t, n) { var r = e.status.total || e.status; if (f(r.hard[n]) <= f(r.used[n])) { @@ -3836,7 +3836,7 @@ target: "_blank" } }, I = function(e, t) { var n = [], r = "Pod" === e.kind ? e : _.get(e, "spec.template"); -return r ? (_.each([ "cpu", "memory", "requests.cpu", "requests.memory", "limits.cpu", "limits.memory", "pods" ], function(a) { +return r && r.spec ? (_.each([ "cpu", "memory", "requests.cpu", "requests.memory", "limits.cpu", "limits.memory", "pods" ], function(a) { var o = t.status.total || t.status; if (("Pod" !== e.kind || "pods" !== a) && _.has(o, [ "hard", a ]) && _.has(o, [ "used", a ])) { var i = P(t, e, a);