fix: wrong value passed to --max-unused when providing a custom prune…

… policy
garethgeorge · Feb 8, 2024 · 34175f2 · 34175f2
1 parent 055ff64
commit 34175f2
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/internal/orchestrator/repo.go b/internal/orchestrator/repo.go
@@ -142,16 +142,17 @@ func (r *RepoOrchestrator) Prune(ctx context.Context, output io.Writer) error {
 	defer r.mu.Unlock()
 
 	policy := r.repoConfig.PrunePolicy
+	if policy == nil {
+		policy = &v1.PrunePolicy{
+			MaxUnusedPercent: 25,
+		}
+	}
 
 	var opts []restic.GenericOption
-	if policy != nil {
-		if policy.MaxUnusedBytes != 0 {
-			opts = append(opts, restic.WithFlags("--max-unused", fmt.Sprintf("%v", policy.MaxUnusedBytes)))
-		} else if policy.MaxUnusedPercent != 0 {
-			opts = append(opts, restic.WithFlags("--max-unused", fmt.Sprintf("%v", policy.MaxUnusedPercent)))
-		}
-	} else {
-		opts = append(opts, restic.WithFlags("--max-unused", "25%"))
+	if policy.MaxUnusedBytes != 0 {
+		opts = append(opts, restic.WithFlags("--max-unused", fmt.Sprintf("%vB", policy.MaxUnusedBytes)))
+	} else if policy.MaxUnusedPercent != 0 {
+		opts = append(opts, restic.WithFlags("--max-unused", fmt.Sprintf("%v%%", policy.MaxUnusedPercent)))
 	}
 
 	r.l.Debug("Prune snapshots")