Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-5030 "lctl {get,set}_param" should also check in /sys/fs/{lnet,lustre}
  3. LU-5330

remove deprecated full path tunable access in lustre config utils

Details

    • Technical task
    • Resolution: Fixed
    • Critical
    • Lustre 2.7.0
    • Lustre 2.7.0
    • 14872

    Description

      lustre/utils/lustre_cfg.c::lprocfs_param_pattern has this warning that needs to be taken care of now:

                      if (!warned) {
                        fprintf(stderr, "%s: specifying parameters via "
                                "full paths is deprecated.\n", cmd);
#if LUSTRE_VERSION_CODE >= OBD_OCD_VERSION(2, 6, 51, 0)
#warning "remove deprecated full path tunable access"
#endif
                        warned = 1;
                }
                snprintf(buf, buf_size, "%s", path);

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-5030 "lctl {get,set}_param" should also check in /sys/fs/{lnet,lustre}

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

          Activity

            [LU-5330] remove deprecated full path tunable access in lustre config utils
            adilger Andreas Dilger added a comment -

            This was fixed for 2.7 as part of http://review.whamcloud.com/9545 LU-4730 utils: lctl get_param, set_param cleanup

            adilger Andreas Dilger added a comment - This was fixed for 2.7 as part of http://review.whamcloud.com/9545 LU-4730 utils: lctl get_param, set_param cleanup
            simmonsja James A Simmons added a comment -

            Actually I will be addressing this with the LU-5030 work. This can be linked to the LU-5030 since this is the case.

            simmonsja James A Simmons added a comment - Actually I will be addressing this with the LU-5030 work. This can be linked to the LU-5030 since this is the case.
            pjones Peter Jones added a comment -

            Nathaniel

            Could you please look into this one?

            Thanks

            Peter

            pjones Peter Jones added a comment - Nathaniel Could you please look into this one? Thanks Peter
            bfaccini Bruno Faccini (Inactive) added a comment - - edited

            I just rebased a patch on top of latest master and triggered this warning/failure during the build :
            https://build.hpdd.intel.com/job/lustre-reviews/25915/

            Seems that we just reached the 2.6.51 version, as indicated by :

            commit 26bc216c8a25db00c4cf3e23bd6cf175610963ea
Author: Oleg Drokin <oleg.drokin@intel.com>
Date:   Fri Aug 15 18:17:13 2014 -0400

    New tag 2.6.51
    
    Change-Id: I45f7812fd7e4d061f2605579c8bac4671a24e202
            bfaccini Bruno Faccini (Inactive) added a comment - - edited I just rebased a patch on top of latest master and triggered this warning/failure during the build : https://build.hpdd.intel.com/job/lustre-reviews/25915/ Seems that we just reached the 2.6.51 version, as indicated by : commit 26bc216c8a25db00c4cf3e23bd6cf175610963ea Author: Oleg Drokin <oleg.drokin@intel.com> Date: Fri Aug 15 18:17:13 2014 -0400 New tag 2.6.51 Change-Id: I45f7812fd7e4d061f2605579c8bac4671a24e202

            People

              utopiabound Nathaniel Clark
              green Oleg Drokin
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: