diff --git a/frontend/public/components/cluster-settings/cluster-settings.tsx b/frontend/public/components/cluster-settings/cluster-settings.tsx index 68913ef8c83..40e581d3dd7 100644 --- a/frontend/public/components/cluster-settings/cluster-settings.tsx +++ b/frontend/public/components/cluster-settings/cluster-settings.tsx @@ -219,7 +219,7 @@ export const CurrentVersion: React.FC = ({ cv }) => { {lastVersion} - + ) : ( <>{t('public~None')} @@ -587,9 +587,8 @@ export const UpdatesGraph: React.FC = ({ cv }) => { const availableUpdates = getSortedAvailableUpdates(cv); const lastVersion = getLastCompletedUpdate(cv); const newestVersion = availableUpdates[0]?.version; - const minorVersionIsNewer = newestVersion - ? isMinorVersionNewer(lastVersion, newestVersion) - : false; + const minorVersionIsNewer = + lastVersion && newestVersion ? isMinorVersionNewer(lastVersion, newestVersion) : false; const secondNewestVersion = availableUpdates[1]?.version; const currentChannel = cv.spec.channel; const currentPrefix = splitClusterVersionChannel(currentChannel)?.prefix; diff --git a/frontend/public/module/k8s/cluster-settings.ts b/frontend/public/module/k8s/cluster-settings.ts index a9ff302902e..8b2fd516953 100644 --- a/frontend/public/module/k8s/cluster-settings.ts +++ b/frontend/public/module/k8s/cluster-settings.ts @@ -85,6 +85,9 @@ export const getSortedNotRecommendedUpdates = (cv: ClusterVersionKind): Conditio export const getNewerMinorVersionUpdate = (currentVersion, availableUpdates) => { const currentVersionParsed = semver.parse(currentVersion); + if (!currentVersionParsed) { + return; + } return availableUpdates?.find( // find the next minor version update, which there should never be more than one (update) => { @@ -101,8 +104,8 @@ export const isMinorVersionNewer = (currentVersion, otherVersion) => { const currentVersionParsed = semver.parse(currentVersion); const otherVersionParsed = semver.parse(otherVersion); return semver.gt( - semver.coerce(`${otherVersionParsed.major}.${otherVersionParsed.minor}`), - semver.coerce(`${currentVersionParsed.major}.${currentVersionParsed.minor}`), + semver.coerce(`${otherVersionParsed?.major}.${otherVersionParsed?.minor}`), + semver.coerce(`${currentVersionParsed?.major}.${currentVersionParsed?.minor}`), ); };