Skip to content

Commit 3727151

Browse files
code refactoring fix
1 parent b7f32d2 commit 3727151

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

pkg/app/AppService.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -989,9 +989,6 @@ func (impl *AppServiceImpl) TriggerRelease(overrideRequest *bean.ValuesOverrideR
989989
var gitCommitStatus pipelineConfig.TimelineStatus
990990
var gitCommitStatusDetail string
991991
err = impl.buildChartAndPushToGitRepo(overrideRequest, ctx, chartMetaData, referenceTemplatePath, wfrId, gitOpsRepoName, envOverride)
992-
if err != nil {
993-
return 0, err
994-
}
995992
if err != nil {
996993
impl.saveTimelineForError(overrideRequest, ctx, err, wfrId)
997994
return 0, err
@@ -1117,13 +1114,12 @@ func (impl *AppServiceImpl) buildChartAndPushToGitRepo(overrideRequest *bean.Val
11171114
span.End()
11181115
defer impl.chartTemplateService.CleanDir(tempReferenceTemplateDir)
11191116
if err != nil {
1120-
impl.saveTimelineForError(overrideRequest, ctx, err, wfrId)
11211117
return err
11221118
}
11231119
_, span = otel.Tracer("orchestrator").Start(ctx, "chartTemplateService.PushChartToGitRepo")
11241120
err = impl.chartTemplateService.PushChartToGitRepo(gitOpsRepoName, envOverride.Chart.ReferenceTemplate, envOverride.Chart.ChartVersion, tempReferenceTemplateDir, envOverride.Chart.GitRepoUrl, overrideRequest.UserId)
11251121
span.End()
1126-
return nil
1122+
return err
11271123
}
11281124

11291125
func (impl *AppServiceImpl) saveTimelineForError(overrideRequest *bean.ValuesOverrideRequest, ctx context.Context, err error, wfrId int) {

0 commit comments

Comments
 (0)