Skip to content

Commit d9cdfb0

Browse files
committed
feat: more verbose errors
1 parent 556b640 commit d9cdfb0

File tree

1 file changed

+22
-0
lines changed

1 file changed

+22
-0
lines changed

internal/app/updater/commit.go

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,8 @@ func commitGitChanges(appName string, gitW git.Worktree, commitMessage string, g
7878
},
7979
})
8080
if err != nil {
81+
logCtx.Errorf(err.Error())
82+
8183
return nil, err
8284
}
8385
return &commit, nil
@@ -93,6 +95,8 @@ func pushGitChanges(appName string, objC object.Commit, gitR *git.Repository, gi
9395
Auth: gitAuth,
9496
})
9597
if err != nil {
98+
logCtx.Errorf(err.Error())
99+
96100
return err
97101
}
98102

@@ -110,6 +114,8 @@ func commitAndPushGitChanges(cfg HelmUpdaterConfig, commitMessage string, gitW g
110114
logCtx.Infof("Adding file %s to git for commit changes", targetFile)
111115
_, err := gitW.Add(targetFile)
112116
if err != nil {
117+
logCtx.Errorf(err.Error())
118+
113119
return err
114120
}
115121

@@ -120,12 +126,16 @@ func commitAndPushGitChanges(cfg HelmUpdaterConfig, commitMessage string, gitW g
120126

121127
gitR, err := git.PlainOpen(tempRoot)
122128
if err != nil {
129+
logCtx.Errorf(err.Error())
130+
123131
return err
124132
}
125133

126134
logCtx.Debugf("Obtaining current HEAD to verify added changes")
127135
obj, err := gitR.CommitObject(*commit)
128136
if err != nil {
137+
logCtx.Errorf(err.Error())
138+
129139
return err
130140
}
131141
err = pushGitChanges(cfg.AppName, *obj, gitR, gitAuth)
@@ -288,26 +298,36 @@ func commitChangesGit(cfg HelmUpdaterConfig, write changeWriter) (*[]ChangeEntry
288298
logCtx := log.WithContext().AddField("application", cfg.AppName)
289299
creds, err := cfg.GitCredentials.NewGitCreds(cfg.GitConf.RepoURL, cfg.GitCredentials.Password)
290300
if err != nil {
301+
logCtx.Errorf(err.Error())
302+
291303
return nil, fmt.Errorf("could not get creds for repo '%s': %v", cfg.AppName, err)
292304
}
293305

294306
tempRoot, err := createTempFileInDirectory(fmt.Sprintf("git-%s", cfg.AppName), cfg.AppName, cfg.GitConf.RepoURL)
295307
if err != nil {
308+
logCtx.Errorf(err.Error())
309+
296310
return nil, err
297311
}
298312

299313
gitW, err := cloneGitRepositoryInBranch(cfg.AppName, cfg.GitConf.RepoURL, creds, *tempRoot, cfg.GitConf.Branch)
300314
if err != nil {
315+
logCtx.Errorf(err.Error())
316+
301317
return nil, err
302318
}
303319

304320
// write changes to files
305321
if apps, err = write(cfg, *tempRoot, *gitW); err != nil {
322+
logCtx.Errorf(err.Error())
323+
306324
return nil, err
307325
}
308326

309327
commitMessage, err := configureCommitMessage(cfg.AppName, apps, cfg.GitConf.Message)
310328
if err != nil {
329+
logCtx.Errorf(err.Error())
330+
311331
return nil, err
312332
}
313333

@@ -318,6 +338,8 @@ func commitChangesGit(cfg HelmUpdaterConfig, write changeWriter) (*[]ChangeEntry
318338

319339
err = commitAndPushGitChanges(cfg, *commitMessage, *gitW, *tempRoot, creds)
320340
if err != nil {
341+
logCtx.Errorf(err.Error())
342+
321343
return nil, err
322344
}
323345

0 commit comments

Comments
 (0)