Skip to content

Commit

Permalink
Note that keys must be loaded for 'zpool remove'
Browse files Browse the repository at this point in the history
The error returned by `zpool remove` when the encryption keys aren't
loaded isn't very helpful.  Furthermore, the man pages make no
mention that the keys need to be loaded. This change doesn't resolve
the error message but it does update the man page to mention this
requirement.

Authored-by: grodik <pat@litke.dev>
Reviewed-by: Matthew Ahrens <mahrens@delphix.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #10939
Closes #10948
  • Loading branch information
geudrik authored and behlendorf committed Oct 1, 2020
1 parent e5a4f9c commit ced5f71
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions man/man8/zpool-remove.8
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,8 @@ This command supports removing hot spare, cache, log, and both mirrored and
non-redundant primary top-level vdevs, including dedup and special vdevs.
When the primary pool storage includes a top-level raidz vdev only hot spare,
cache, and log devices can be removed.
Note that keys for all encrypted datasets must be loaded for top-level vdevs
to be removed.
.sp
Removing a top-level vdev reduces the total amount of space in the storage pool.
The specified device will be evacuated by copying all allocated space from it to
Expand Down

0 comments on commit ced5f71

Please sign in to comment.