Skip to content

Commit

Permalink
fix: return true if chart install succeeds
Browse files Browse the repository at this point in the history
Signed-off-by: Lenin Mehedy <[email protected]>
  • Loading branch information
leninmehedy committed Nov 16, 2023
1 parent 62ae48e commit bb31e0d
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions fullstack-network-manager/src/commands/chart.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,13 @@ export class ChartCommand extends BaseCommand {
await this.chartManager.install(namespace, constants.FST_CHART_DEPLOYMENT_NAME, chartPath, config.version, valuesArg)

this.logger.showList('charts', await this.chartManager.getInstalledCharts(namespace))

return true
} catch (e) {
this.logger.showUserError(e)
}

return false
}

async uninstall (argv) {
Expand Down

0 comments on commit bb31e0d

Please sign in to comment.