Skip to content

Commit

Permalink
rearrange plan for deletion
Browse files Browse the repository at this point in the history
  • Loading branch information
petedannemann committed Dec 12, 2023
1 parent 6843aa6 commit aadd135
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/acl/acl.go
Original file line number Diff line number Diff line change
Expand Up @@ -166,15 +166,15 @@ func (a *ACLAdmin) Delete(ctx context.Context, filter kafka.DeleteACLsFilter) er
return fmt.Errorf("No ACL matches filter:\n%+v", formatACLs(filter))
}

log.Infof("%d ACLs in the cluster match the filter provided", len(clusterACLs))
log.Infof("The following ACLs in the cluster are planned for deletion:\n%+v", formatACLInfos(clusterACLs))

if a.config.DryRun {
log.Infof("Would delete ACLs:\n%+v", formatACLInfos(clusterACLs))
return nil
}

// This isn't settable by the CLI for safety measures but allows for testability
confirm, err := util.Confirm(fmt.Sprintf("Delete ACLs?\n%+v", formatACLInfos(clusterACLs)), a.config.SkipConfirm)
confirm, err := util.Confirm("Delete ACLs?", a.config.SkipConfirm)
if err != nil {
return err
}
Expand Down

0 comments on commit aadd135

Please sign in to comment.