Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.16.0, Lustre 2.15.0
    • None
    • None
    • 9223372036854775807

    Description

      Remove an obsolete lctl command ("lov_getconfig"), as well as the ioctl that it was calling. It was only recently used by sanity test_44a, which can instead use "lfs getstripe -d $DIR" now that LU-11656 is landed.

      The lctl command list should also be reordered so that the "testing (DANGEROUS)" commands are listed at the end, and LFSCK and llog commands are before that, to avoid confusion.

      There are also a number of very ancient jt_ptl_ and jt_obd_ function definitions and ancient ioctl numbers defined that are no longer used and can be removed.

      Attachments

        Issue Links

          Activity

            [LU-13107] clean up lctl commands
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.16.0 [ 15190 ]
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-16634 [ LU-16634 ]
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-17489 [ LU-17489 ]

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48886/
            Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: ccee6b92ec4d338f54df0ebb2e8533b870265e4b

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48886/ Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg Project: fs/lustre-release Branch: master Current Patch Set: Commit: ccee6b92ec4d338f54df0ebb2e8533b870265e4b

            "Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48886
            Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 44c558d9b540c9f2d106a72f76559c044537346c

            gerrit Gerrit Updater added a comment - "Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48886 Subject: LU-13107 utils: remove duplicate lctl erase/fork_lcfg Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 44c558d9b540c9f2d106a72f76559c044537346c
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.15.0 [ 14791 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            pjones Peter Jones added a comment -

            Seems to have all landed for 2.15

            pjones Peter Jones added a comment - Seems to have all landed for 2.15

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37108/
            Subject: LU-13107 utils: clean up lctl command usage
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: b0efebdaef52d8ac9b02857166ceb00079612ebc

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37108/ Subject: LU-13107 utils: clean up lctl command usage Project: fs/lustre-release Branch: master Current Patch Set: Commit: b0efebdaef52d8ac9b02857166ceb00079612ebc

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37107/
            Subject: LU-13107 uapi: remove obsolete ioctls
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 0f38a0b9db4f7c6c101ee55fe7b94d49a32eabe9

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37107/ Subject: LU-13107 uapi: remove obsolete ioctls Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0f38a0b9db4f7c6c101ee55fe7b94d49a32eabe9

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37106/
            Subject: LU-13107 utils: remove lctl lov_getconfig command
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: d2cb789537485eec9bfc5bc7d4c85025b0488478

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/37106/ Subject: LU-13107 utils: remove lctl lov_getconfig command Project: fs/lustre-release Branch: master Current Patch Set: Commit: d2cb789537485eec9bfc5bc7d4c85025b0488478

            People

              adilger Andreas Dilger
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: