improving logging with git status for auto-bump

Author: nicholasSUSE

pkg/auto/chart_bump.go

@@ -240,6 +240,11 @@ func (b *Bump) BumpChart(ctx context.Context) error {
 		return err
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	if err := git.AddAndCommit("make prepare"); err != nil {
 		logger.Log(ctx, slog.LevelError, "error while adding and committing after make prepare", logger.Err(err))
 		return err
@@ -253,6 +258,11 @@ func (b *Bump) BumpChart(ctx context.Context) error {
 		}
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	if clean, _ := git.StatusProcelain(ctx); !clean {
 		logger.Log(ctx, slog.LevelDebug, "git is not clean - icon downloaded")
 		if err := git.AddAndCommit("make icon"); err != nil {
@@ -272,6 +282,11 @@ func (b *Bump) BumpChart(ctx context.Context) error {
 		return err
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	if clean, _ := git.StatusProcelain(ctx); !clean {
 		if err := git.AddAndCommit("make patch"); err != nil {
 			return err
@@ -284,12 +299,22 @@ func (b *Bump) BumpChart(ctx context.Context) error {
 		return err
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	// make charts - generate new assets and charts overwriting logo
 	if err := b.Pkg.GenerateCharts(ctx, b.configOptions.OmitBuildMetadataOnExport); err != nil {
 		logger.Log(ctx, slog.LevelError, "error while generating charts", logger.Err(err))
 		return err
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	if clean, _ := git.StatusProcelain(ctx); clean {
 		logger.Log(ctx, slog.LevelError, "make charts did not generate any changes")
 		return errors.New("make charts did not generate any changes")
@@ -318,12 +343,22 @@ func (b *Bump) BumpChart(ctx context.Context) error {
 		}
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	// modify the release.yaml
 	if err := b.updateReleaseYaml(ctx, targetCharts); err != nil {
 		logger.Log(ctx, slog.LevelError, "error while updating release.yaml", logger.Err(err))
 		return err
 	}
 
+	if err := git.Status(ctx); err != nil {
+		logger.Log(ctx, slog.LevelError, "error while checking git status", logger.Err(err))
+		return err
+	}
+
 	if clean, _ := git.StatusProcelain(ctx); clean {
 		logger.Log(ctx, slog.LevelError, "update release.yaml did not generate any changes")
 		return errors.New("update release.yaml did not generate any changes")