[LU-3155] Permanent parameters with lctl set_param -P

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.5.0
• Affects Version/s: Lustre 2.5.0
• Labels: patch
• Assignee: Emoly Liu
• Reporter: Artem Blagodarenko (Inactive)

Issue Links

• is duplicated by: LU-2629 set_param and conf_param have different syntaxes
• is related to: LU-3798 replay-single test_86: configuration log errors
• is related to: LUDOC-166 Document new "lctl set_param -P" option
• is related to: LU-7004 fix "lctl set_param -P" to allow deprecation of "lctl conf_param"

Comments

[Artem Blagodarenko (Inactive)]

>Artem, Please take a look at LU-3798 and comment.
Thanks. Done.

[Bob Glossman (Inactive)]

Artem, Please take a look at LU-3798 and comment.

[Artem Blagodarenko (Inactive)]

Warnings and man page
http://review.whamcloud.com/7452
http://review.whamcloud.com/#/c/7436/

[Andreas Dilger]

I noticed that the original patch is also missing an update to the lustre/doc/lctl.8 man page.

[Emoly Liu]

patch http://review.whamcloud.com/6025 landed for 2.5

[Andreas Dilger]

It would be useful to have two additional patches for this change:

• add deprecation warnings for "conf_param" usage for a future release (e.g. LUSTRE_VERSION_CODE > OBD_OCD_VERSION(2, 7, 53, 0)) and disable the code entirely in some future release (e.g. > 3.0.53)
• remove the use of "lctl conf_param" from test scripts around 2.7.53 (want to keep these for a while for interop testing)

[Andreas Dilger]

I filed LUDOC-166 for tracking the required documentation update for this feature.

[Artem Blagodarenko (Inactive)]

Bug is fixed. Waiting for testing results.

[Artem Blagodarenko (Inactive)]

Test always fails on test_132 of sanity with message 'som is still disabled'.
As I understand som is enabled on MGS and checked on MDS (this servers are on different hosts) and I can't find code that offer check this setting on MDS. Should this scheme works?

Indeed, this test works on other branches. I can't understand why and how my patch blokes this.

[Emoly Liu]

The patch that Artem pushed is always hitting some test failures. After the patch passes the test and review, it will be landed.

[Nathan Rutman]

Can we get this landed please? I really would like to see a simpler and more robust configuration, and we don't want to diverge our branch from mainstream too far.