Merge pull request #8505 from steven-zou/fix/retention_bug_dryrun

fix bug: dry run is missing when deleting ropo
This commit is contained in:
Steven Zou 2019-07-31 18:40:22 +08:00 committed by GitHub
commit 35f71d1bc6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -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