summaryrefslogtreecommitdiffstats
path: root/cmd
diff options
context:
space:
mode:
authorTim Connors <[email protected]>2013-02-26 08:00:45 +1100
committerBrian Behlendorf <[email protected]>2013-02-28 09:17:09 -0800
commitc5b247f3354d5c9e24577af997c84d90625620a6 (patch)
tree9ff534477bb8dbfb4747e9b5b989697138f21ed9 /cmd
parentd9b0ebbe824469d178a05b0fb9004e4afce86009 (diff)
-x shouldn't warn about old on-disk format or unavailable features
`zpool status -x` should only flag errors or where the pool is unavailable. If it imported fine but isn't using the latest features available in the code, that's not an error. Signed-off-by: Brian Behlendorf <[email protected]> Closes #1319
Diffstat (limited to 'cmd')
-rw-r--r--cmd/zpool/zpool_main.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/cmd/zpool/zpool_main.c b/cmd/zpool/zpool_main.c
index c52bd3ed7..320069873 100644
--- a/cmd/zpool/zpool_main.c
+++ b/cmd/zpool/zpool_main.c
@@ -3970,7 +3970,10 @@ status_callback(zpool_handle_t *zhp, void *data)
* If we were given 'zpool status -x', only report those pools with
* problems.
*/
- if (reason == ZPOOL_STATUS_OK && cbp->cb_explain) {
+ if (cbp->cb_explain &&
+ (reason == ZPOOL_STATUS_OK ||
+ reason == ZPOOL_STATUS_VERSION_OLDER ||
+ reason == ZPOOL_STATUS_FEAT_DISABLED)) {
if (!cbp->cb_allpools) {
(void) printf(gettext("pool '%s' is healthy\n"),
zpool_get_name(zhp));