Whamcloud - gitweb
LU-9544 utils: set -P if missing in 'set_param -d' 99/55399/18
authorFrederick Dilger <fdilger@whamcloud.com>
Tue, 11 Jun 2024 21:35:35 +0000 (17:35 -0400)
committerOleg Drokin <green@whamcloud.com>
Wed, 31 Jul 2024 16:02:55 +0000 (16:02 +0000)
commitd7f8c21eed5ba232ae48863001b512c207427037
tree39b885092f4c17d8f5a98c16b0df80188fdf95cf
parent4fbcf088fdb0e0553eab45b3ba4caf84518d7b7e
LU-9544 utils: set -P if missing in 'set_param -d'

The -P option to lctl set_param will now be added if
the -d option (for delete) is specified by itself.

As described in the ticket, if a value is erroneously supplied when
using -P and -d then instead of being deleted, the parameter is
set to the old value with a trailing '='. A non-regression test
has been created to verify that this isn't happening.

wait_update_cond and wait_update were modified to avoid unresolved
errors from throwing an unwanted errnum during the wait. They still
operate with logical equivalence to before.

Signed-off-by: Frederick Dilger <fdilger@whamcloud.com>
Change-Id: If35b2e9db51f7296da25b798205b9f9104830bca
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/55399
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
lustre/tests/sanity.sh
lustre/tests/test-framework.sh
lustre/utils/lustre_cfg.c