Merge pull request #214 from nicholasSUSE/check-to-bump-version

check upstream app version

Author: nicholasSUSE

pkg/auto/chart_bump.go

@@ -245,6 +245,16 @@ func (b *Bump) BumpChart(ctx context.Context, versionOverride string) error {
 		return err
 	}
 
+	// check if the version to bump does not already exists
+	alreadyExist, err := checkBumpAppVersion(ctx, b.Pkg.UpstreamChartVersion, b.assetsVersionsMap[b.targetChart])
+	if err != nil {
+		return err
+	}
+	if alreadyExist {
+		git.FullReset() // quitting the job regardless if this works or not
+		return errors.New("version to bump already exists: " + *b.Pkg.UpstreamChartVersion)
+	}
+
 	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
@@ -507,3 +517,23 @@ func writeBumpJSON(targetCharts []string, bumpVersion string) error {
 
 	return nil
 }
+
+// checkBumpAppVersion checks if the bumpAppVersion already exists in the repository
+func checkBumpAppVersion(ctx context.Context, bumpAppVersion *string, versions []lifecycle.Asset) (bool, error) {
+	if bumpAppVersion == nil {
+		logger.Log(ctx, slog.LevelError, "upstreamVersion is nil for chart, abnormal behavior")
+		return false, errors.New("upstreamVersion is nil for chart, abnormal behavior")
+	}
+
+	for _, version := range versions {
+		parts := strings.Split(version.Version, "+up")
+		if len(parts) != 2 {
+			continue
+		}
+		if parts[1] == *bumpAppVersion {
+			return true, nil
+		}
+	}
+
+	return false, nil
+}