From 49c1b2d4f1f934225988f32eeb373953f84399e4 Mon Sep 17 00:00:00 2001 From: Steven Zou Date: Wed, 31 Jul 2019 17:14:08 +0800 Subject: [PATCH] fix bug: dry run is missing when deleting ropo Signed-off-by: Steven Zou --- src/pkg/retention/job_del_repo.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/pkg/retention/job_del_repo.go b/src/pkg/retention/job_del_repo.go index ccf839d1f..ab8a5f535 100644 --- a/src/pkg/retention/job_del_repo.go +++ b/src/pkg/retention/job_del_repo.go @@ -74,8 +74,10 @@ func (drj *DelRepoJob) Run(ctx job.Context, params job.Parameters) error { } // Delete the repository - if err := dep.DefaultClient.DeleteRepository(repo); err != nil { - return err + if !isDryRun { + if err := dep.DefaultClient.DeleteRepository(repo); err != nil { + return err + } } // Log deletions