Skip to content

Commit f5edc85

Browse files
vishesh92dhslove
authored andcommitted
Fixup CKS UI for external managed clusters (apache#9434)
1 parent 9db5099 commit f5edc85

File tree

3 files changed

+7
-8
lines changed

3 files changed

+7
-8
lines changed

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
@@ -1626,8 +1626,7 @@ public boolean upgradeKubernetesCluster(UpgradeKubernetesClusterCmd cmd) throws
16261626
private void updateNodeCount(KubernetesClusterVO kubernetesCluster) {
16271627
List<KubernetesClusterVmMapVO> nodeList = kubernetesClusterVmMapDao.listByClusterId(kubernetesCluster.getId());
16281628
kubernetesCluster.setControlNodeCount(nodeList.stream().filter(KubernetesClusterVmMapVO::isControlNode).count());
1629-
kubernetesCluster.setNodeCount(nodeList.size());
1630-
kubernetesCluster.setNodeCount(nodeList.size());
1629+
kubernetesCluster.setNodeCount(nodeList.size() - kubernetesCluster.getControlNodeCount());
16311630
kubernetesClusterDao.persist(kubernetesCluster);
16321631
}
16331632

ui/src/components/view/ListView.vue

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,9 +96,6 @@
9696
</span>
9797
</span>
9898
</template>
99-
<template v-if="record.clustertype === 'ExternalManaged' && $route.path.split('/')[1] === 'kubernetes' && ['cpunumber', 'memory', 'size'].includes(column.key)">
100-
<span>{{ text <= 0 ? 'N/A' : text }}</span>
101-
</template>
10299
<template v-if="column.key === 'templatetype'">
103100
<span>{{ text }}</span>
104101
</template>
@@ -177,7 +174,10 @@
177174
<router-link v-if="resourceIdToValidLinksMap[record.id]?.volume" :to="{ path: '/volume/' + record.volumeid }">{{ text }}</router-link>
178175
<span v-else>{{ text }}</span>
179176
</template>
180-
<template v-if="column.key === 'size'">
177+
<template v-if="record.clustertype === 'ExternalManaged' && $route.path.split('/')[1] === 'kubernetes' && ['kubernetesversionname', 'cpunumber', 'memory', 'size'].includes(column.key)">
178+
<span>{{ text <= 0 || !text ? 'N/A' : text }}</span>
179+
</template>
180+
<template v-else-if="column.key === 'size'">
181181
<span v-if="text && $route.path === '/kubernetes'">
182182
{{ text }}
183183
</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)