Skip to content

Commit

Permalink
rpk: bugfix - filter in list-mount
Browse files Browse the repository at this point in the history
We should default to filter `all`, otherwise we
won't print any operation unless a filter is
provided.
  • Loading branch information
r-vasquez committed Nov 15, 2024
1 parent 26e5aa3 commit 72cad3a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/go/rpk/pkg/cli/cluster/storage/cancel-mount.go
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ Cancel a mount/unmount operation
out.MaybeDie(err, "unable to cancel the mount/unmount operation: %v", err)
}

fmt.Printf("Successfully canceled the operation with ID %v", migrationID)
fmt.Printf("Successfully canceled the operation with ID %v\n", migrationID)
},
}
return cmd
Expand Down
2 changes: 1 addition & 1 deletion src/go/rpk/pkg/cli/cluster/storage/list-mount.go
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ Use filter to list only migrations in a specific state
},
}
p.InstallFormatFlag(cmd)
cmd.Flags().StringVarP(&filter, "filter", "f", "", "Filter the list of migrations by state. Only valid for text")
cmd.Flags().StringVarP(&filter, "filter", "f", "all", "Filter the list of migrations by state. Only valid for text")
return cmd
}

Expand Down

0 comments on commit 72cad3a

Please sign in to comment.