Skip to content

Commit 674fc28

Browse files
authored
Merge pull request #235 from devtron-labs/fix-panic
fix: async mode change panic fix
2 parents ae32c1e + 09d88de commit 674fc28

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

kubelink/pkg/service/helmApplicationService/helmAppService.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -736,7 +736,7 @@ func (impl *HelmAppServiceImpl) installRelease(ctx context.Context, request *cli
736736
chartSpec.CreateNamespace = false
737737

738738
impl.logger.Debugw("Installing release", "name", releaseIdentifier.ReleaseName, "namespace", releaseIdentifier.ReleaseNamespace, "dry-run", dryRun)
739-
if runInstallInAsyncMode(request.InstallAppVersionHistoryId, impl.helmReleaseConfig.RunHelmInstallInAsyncMode) {
739+
if !runInstallInAsyncMode(request.InstallAppVersionHistoryId, impl.helmReleaseConfig.RunHelmInstallInAsyncMode) {
740740
impl.logger.Debugw("Installing release", "name", releaseIdentifier.ReleaseName, "namespace", releaseIdentifier.ReleaseNamespace, "dry-run", dryRun)
741741
rel, err := helmClientObj.InstallChart(context.Background(), chartSpec)
742742
if err != nil {
@@ -849,7 +849,7 @@ func (impl *HelmAppServiceImpl) UpgradeReleaseWithChartInfo(ctx context.Context,
849849
chartSpec.DependencyUpdate = true
850850
chartSpec.UpgradeCRDs = true
851851

852-
if runInstallInAsyncMode(request.InstallAppVersionHistoryId, impl.helmReleaseConfig.RunHelmInstallInAsyncMode) {
852+
if !runInstallInAsyncMode(request.InstallAppVersionHistoryId, impl.helmReleaseConfig.RunHelmInstallInAsyncMode) {
853853
impl.logger.Debug("Upgrading release with chart info")
854854
_, err = helmClientObj.UpgradeReleaseWithChartInfo(context.Background(), chartSpec)
855855
if UpgradeErr, ok := err.(*driver.StorageDriverError); ok {

0 commit comments

Comments
 (0)