Skip to content

Commit 075051d

Browse files
committed
removed todo in title: 'Check if cluster setup chart is installed',
Signed-off-by: Jeromy Cannon <[email protected]>
1 parent ad56068 commit 075051d

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

src/commands/network.ts

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -646,14 +646,18 @@ export class NetworkCommand extends BaseCommand {
646646
},
647647
{
648648
title: 'Check if cluster setup chart is installed',
649-
task: async () => {
650-
// TODO @Lenin, need to use the config.contexts for each isChartInstalled call
651-
const isChartInstalled = await this.chartManager.isChartInstalled(null, constants.SOLO_CLUSTER_SETUP_CHART);
652-
if (!isChartInstalled) {
653-
// TODO @Lenin, update error message
654-
throw new SoloError(
655-
`Chart ${constants.SOLO_CLUSTER_SETUP_CHART} is not installed. Run 'solo cluster setup'`,
649+
task: async ctx => {
650+
for (const context of ctx.config.contexts) {
651+
const isChartInstalled = await this.chartManager.isChartInstalled(
652+
null,
653+
constants.SOLO_CLUSTER_SETUP_CHART,
654+
context,
656655
);
656+
if (!isChartInstalled) {
657+
throw new SoloError(
658+
`Chart ${constants.SOLO_CLUSTER_SETUP_CHART} is not installed for cluster: ${context}. Run 'solo cluster setup'`,
659+
);
660+
}
657661
}
658662
},
659663
},

0 commit comments

Comments
 (0)