delete the repository if codeowners file failed to be added

Author: Skarlso

pkg/providers/gitea/gitea.go

@@ -5,9 +5,11 @@
 package gitea
 
 import (
+	"bytes"
 	"context"
+	"encoding/base64"
+	"errors"
 	"fmt"
-	"strings"
 
 	"code.gitea.io/sdk/gitea"
 	v1 "k8s.io/api/core/v1"
@@ -90,21 +92,28 @@ func (c *Client) CreateRepository(ctx context.Context, obj mpasv1alpha1.Reposito
 	}
 
 	if len(obj.Spec.Maintainers) != 0 {
-		content := strings.Builder{}
+		var content []byte
+		buffer := bytes.NewBuffer(content)
 
 		for _, m := range obj.Spec.Maintainers {
-			_, _ = content.WriteString(fmt.Sprintf("%s\n", m))
+			buffer.WriteString(m)
 		}
 
+		encoded := base64.StdEncoding.EncodeToString(buffer.Bytes())
+
 		_, _, err := client.CreateFile(obj.Spec.Owner, obj.Spec.RepositoryName, "CODEOWNERS", gitea.CreateFileOptions{
 			FileOptions: gitea.FileOptions{
 				Message:    "Adding CODEOWNERS file.",
 				BranchName: "main",
 			},
-			Content: content.String(),
+			Content: encoded,
 		})
 
 		if err != nil {
+			if _, derr := client.DeleteRepo(obj.Spec.Owner, obj.Spec.RepositoryName); derr != nil {
+				err = errors.Join(err, derr)
+			}
+
 			return fmt.Errorf("failed to add CODEOWNERS file: %w", err)
 		}
 	}

pkg/providers/gogit/gogit.go

@@ -6,6 +6,7 @@ package gogit
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"strings"
 
@@ -51,6 +52,10 @@ func CreateOrganizationRepository(ctx context.Context, gc gitprovider.Client, do
 		}
 
 		if err := createCodeownersFile(ctx, repo, spec.Maintainers); err != nil {
+			if cerr := repo.Delete(ctx); cerr != nil {
+				err = errors.Join(err, cerr)
+			}
+
 			return fmt.Errorf("failed to add CODEOWNERS file: %w", err)
 		}
 
@@ -65,6 +70,10 @@ func CreateOrganizationRepository(ctx context.Context, gc gitprovider.Client, do
 			logger.Info("using existing repository", "domain", domain, "repository", spec.RepositoryName)
 		} else {
 			if err := createCodeownersFile(ctx, repo, spec.Maintainers); err != nil {
+				if cerr := repo.Delete(ctx); cerr != nil {
+					err = errors.Join(err, cerr)
+				}
+
 				return fmt.Errorf("failed to add CODEOWNERS file: %w", err)
 			}
 
@@ -112,6 +121,10 @@ func CreateUserRepository(ctx context.Context, gc gitprovider.Client, domain str
 		}
 
 		if err := createCodeownersFile(ctx, repo, spec.Maintainers); err != nil {
+			if cerr := repo.Delete(ctx); cerr != nil {
+				err = errors.Join(err, cerr)
+			}
+
 			return fmt.Errorf("failed to add CODEOWNERS file: %w", err)
 		}
 
@@ -126,6 +139,10 @@ func CreateUserRepository(ctx context.Context, gc gitprovider.Client, domain str
 			logger.Info("using existing repository", "domain", domain, "repository", spec.RepositoryName)
 		} else {
 			if err := createCodeownersFile(ctx, repo, spec.Maintainers); err != nil {
+				if cerr := repo.Delete(ctx); cerr != nil {
+					err = errors.Join(err, cerr)
+				}
+
 				return fmt.Errorf("failed to add CODEOWNERS file: %w", err)
 			}