Skip to content

Commit 0be87ac

Browse files
authored
Fixup CKS UI for external managed clusters (#9434)
1 parent 3d32ab5 commit 0be87ac

3 files changed

Lines changed: 7 additions & 8 deletions

File tree

plugins/integrations/kubernetes-service/src/main/java/com/cloud/kubernetes/cluster/KubernetesClusterManagerImpl.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1625,8 +1625,7 @@ public boolean upgradeKubernetesCluster(UpgradeKubernetesClusterCmd cmd) throws
16251625
private void updateNodeCount(KubernetesClusterVO kubernetesCluster) {
16261626
List<KubernetesClusterVmMapVO> nodeList = kubernetesClusterVmMapDao.listByClusterId(kubernetesCluster.getId());
16271627
kubernetesCluster.setControlNodeCount(nodeList.stream().filter(KubernetesClusterVmMapVO::isControlNode).count());
1628-
kubernetesCluster.setNodeCount(nodeList.size());
1629-
kubernetesCluster.setNodeCount(nodeList.size());
1628+
kubernetesCluster.setNodeCount(nodeList.size() - kubernetesCluster.getControlNodeCount());
16301629
kubernetesClusterDao.persist(kubernetesCluster);
16311630
}
16321631

ui/src/components/view/ListView.vue

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,6 @@
9292
</span>
9393
</span>
9494
</template>
95-
<template v-if="record.clustertype === 'ExternalManaged' && $route.path.split('/')[1] === 'kubernetes' && ['cpunumber', 'memory', 'size'].includes(column.key)">
96-
<span>{{ text <= 0 ? 'N/A' : text }}</span>
97-
</template>
9895
<template v-if="column.key === 'templatetype'">
9996
<span>{{ text }}</span>
10097
</template>
@@ -170,7 +167,10 @@
170167
<router-link v-if="resourceIdToValidLinksMap[record.id]?.volume" :to="{ path: '/volume/' + record.volumeid }">{{ text }}</router-link>
171168
<span v-else>{{ text }}</span>
172169
</template>
173-
<template v-if="column.key === 'size'">
170+
<template v-if="record.clustertype === 'ExternalManaged' && $route.path.split('/')[1] === 'kubernetes' && ['kubernetesversionname', 'cpunumber', 'memory', 'size'].includes(column.key)">
171+
<span>{{ text <= 0 || !text ? 'N/A' : text }}</span>
172+
</template>
173+
<template v-else-if="column.key === 'size'">
174174
<span v-if="text && $route.path === '/kubernetes'">
175175
{{ text }}
176176
</span>

ui/src/views/compute/KubernetesServiceTab.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
<a-tab-pane :tab="$t('label.details')" key="details">
2626
<DetailsTab :resource="resource" :loading="loading" />
2727
</a-tab-pane>
28-
<a-tab-pane v-if="resource.clustertype == 'CloudManaged'" :tab="$t('label.access')" key="access">
28+
<a-tab-pane v-if="resource.clustertype === 'CloudManaged'" :tab="$t('label.access')" key="access">
2929
<a-card :title="$t('label.kubeconfig.cluster')" :loading="versionLoading">
3030
<div v-if="clusterConfig !== ''">
3131
<a-textarea :value="clusterConfig" :rows="5" readonly />
@@ -278,7 +278,7 @@ export default {
278278
}
279279
},
280280
mounted () {
281-
if (this.$store.getters.apis.scaleKubernetesCluster.params.filter(x => x.name === 'nodeids').length > 0) {
281+
if (this.$store.getters.apis.scaleKubernetesCluster.params.filter(x => x.name === 'nodeids').length > 0 && this.resource.clustertype === 'CloudManaged') {
282282
this.vmColumns.push({
283283
key: 'actions',
284284
title: this.$t('label.actions'),

0 commit comments

Comments
 (0)