Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-3155

Permanent parameters with lctl set_param -P

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.5.0
    • Lustre 2.5.0
    • 7649

    Attachments

      Issue Links

        is duplicated by

        Improvement - An improvement or enhancement to an existing feature or task. LU-2629 set_param and conf_param have different syntaxes

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Resolved
        is related to

        Bug - A problem which impairs or prevents the functions of the product. LU-3798 replay-single test_86: configuration log errors

        • Major - Major loss of function.
        • Resolved

        Bug - A problem which impairs or prevents the functions of the product. LUDOC-166 Document new "lctl set_param -P" option

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Resolved

        Improvement - An improvement or enhancement to an existing feature or task. LU-7004 fix "lctl set_param -P" to allow deprecation of "lctl conf_param"

        • Critical - Crashes, loss of data, severe memory leak.
        • Resolved

        Activity

          [LU-3155] Permanent parameters with lctl set_param -P
          emoly.liu Emoly Liu added a comment -

          Warnings and man page change landed for 2.5

          emoly.liu Emoly Liu added a comment - Warnings and man page change landed for 2.5
          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment - - edited

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

          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment - - edited >Artem, Please take a look at LU-3798 and comment. Thanks. Done.
          bogl Bob Glossman (Inactive) added a comment -

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

          bogl Bob Glossman (Inactive) added a comment - Artem, Please take a look at LU-3798 and comment.
          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment -

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

          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment - Warnings and man page http://review.whamcloud.com/7452 http://review.whamcloud.com/#/c/7436/
          adilger Andreas Dilger added a comment -

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

          adilger Andreas Dilger added a comment - I noticed that the original patch is also missing an update to the lustre/doc/lctl.8 man page.
          emoly.liu Emoly Liu added a comment -

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

          emoly.liu Emoly Liu added a comment - patch http://review.whamcloud.com/6025 landed for 2.5
          adilger Andreas Dilger added a comment -

          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)
          adilger Andreas Dilger added a comment - 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)
          adilger Andreas Dilger added a comment -

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

          adilger Andreas Dilger added a comment - I filed LUDOC-166 for tracking the required documentation update for this feature.
          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment -

          Bug is fixed. Waiting for testing results.

          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment - Bug is fixed. Waiting for testing results.
          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment -

          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.

          artem_blagodarenko Artem Blagodarenko (Inactive) added a comment - 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.

          People

            emoly.liu Emoly Liu
            artem_blagodarenko Artem Blagodarenko (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            11 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: