Skip to content

Commit e070ba2

Browse files
authored
Merge pull request #6301 from devtron-labs/env-override-fix
misc: nil check in envOverride.chart
2 parents eae4ea6 + 60c9edd commit e070ba2

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/deployment/manifest/ManifestCreationService.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ import (
3838
bean3 "github.com/devtron-labs/devtron/pkg/deployment/manifest/bean"
3939
"github.com/devtron-labs/devtron/pkg/deployment/manifest/deployedAppMetrics"
4040
"github.com/devtron-labs/devtron/pkg/deployment/manifest/deploymentTemplate"
41+
"github.com/devtron-labs/devtron/pkg/deployment/manifest/deploymentTemplate/adapter"
4142
bean2 "github.com/devtron-labs/devtron/pkg/deployment/manifest/deploymentTemplate/bean"
4243
"github.com/devtron-labs/devtron/pkg/deployment/manifest/deploymentTemplate/chartRef"
4344
"github.com/devtron-labs/devtron/pkg/deployment/manifest/deploymentTemplate/read"
@@ -473,10 +474,9 @@ func (impl *ManifestCreationServiceImpl) getEnvOverrideForLastSavedConfigTrigger
473474
impl.logger.Errorw("error in creating envConfig", "data", envOverride, "error", err)
474475
return nil, err
475476
}
477+
envOverride = adapter.EnvOverrideDBToDTO(envOverrideDBObj)
476478
}
477-
if envOverride != nil {
478-
envOverride.Chart = chart
479-
}
479+
envOverride.Chart = chart
480480
} else if envOverride.Id > 0 && !envOverride.IsOverride {
481481
_, span = otel.Tracer("orchestrator").Start(ctx, "chartRepository.FindLatestChartForAppByAppId")
482482
chart, err = impl.chartRepository.FindLatestChartForAppByAppId(overrideRequest.AppId)

0 commit comments

Comments
 (0)