Whamcloud - gitweb
LU-7004 utils: remove "lctl conf_param" deprecation 73/20573/2
authorAndreas Dilger <andreas.dilger@intel.com>
Thu, 2 Jun 2016 01:02:09 +0000 (19:02 -0600)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 20 Jun 2016 00:58:44 +0000 (00:58 +0000)
The "lctl set_param -P" functionality was intended to replace
"lctl conf_param" with a more uniform interface that matches
the existing "lctl set_param" and "lctl get_param" usage.
However, there are still unresolved issues that need to be
addressed and a longer transition period needed before the
old functionality can be removed.

Remove the deprecation warning for now, rather than bumping it
to some later release.  It can be re-added when "set_param -P"
has been fixed.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Change-Id: If9a316bda4bf8d889c4072740f0879d9d43ebbe5
Reviewed-on: http://review.whamcloud.com/20573
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bob Glossman <bob.glossman@intel.com>
Reviewed-by: James Nunez <james.a.nunez@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>

index 7e859c1..bdd6c8e 100644 (file)
@@ -617,11 +617,6 @@ int jt_lcfg_mgsparam(int argc, char **argv)
        struct lustre_cfg *lcfg;
        char *buf = NULL;
-       fprintf(stderr, "warning: 'lctl conf_param' is deprecated, "
-               "use 'lctl set_param -P' instead\n");
         /* mgs_setparam processes only lctl buf #1 */
         if ((argc > 3) || (argc <= 1))
                 return CMD_HELP;