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

Update lctl man page to include LFSCK 2 Changes

Details

    • Improvement
    • Resolution: Fixed
    • Critical
    • Lustre 2.6.0
    • Lustre 2.6.0
    • 13416

    Description

      The man page for lctl needs to be updated to include changes to lfsck_start and lfsck_stop from the LFSCK Phase 2 work.

      At the least, the -A and -o options need to be added to lfsck_start and -A to lfsck_stop, but the explanation of the options also need to be updated.

      Attachments

        Activity

          [LU-4862] Update lctl man page to include LFSCK 2 Changes

          Patch 10104 was abandoned because http://review.whamcloud.com/#/c/10091 incorporates all of these changes.

          jamesanunez James Nunez (Inactive) added a comment - Patch 10104 was abandoned because http://review.whamcloud.com/#/c/10091 incorporates all of these changes.

          In patch 9910, a comment requested that the lfsck_start --windows option be changed to --window. That change is in the patch at http://review.whamcloud.com/10104 .

          Only the lfsck_start option was changed. Internal use of "windows" was not changed for routine names, variables or constants.

          jamesanunez James Nunez (Inactive) added a comment - In patch 9910, a comment requested that the lfsck_start --windows option be changed to --window. That change is in the patch at http://review.whamcloud.com/10104 . Only the lfsck_start option was changed. Internal use of "windows" was not changed for routine names, variables or constants.

          The updated lctl man page patch at http://review.whamcloud.com/9910

          jamesanunez James Nunez (Inactive) added a comment - The updated lctl man page patch at http://review.whamcloud.com/9910

          That patch is for the Lustre manual. The lustre/doc/lctl.8 man page should also be updated to describe these new options.

          adilger Andreas Dilger added a comment - That patch is for the Lustre manual. The lustre/doc/lctl.8 man page should also be updated to describe these new options.
          yong.fan nasf (Inactive) added a comment - This one? http://review.whamcloud.com/#/c/9068/

          Is there already a patch for this somewhere? I thought there was.

          adilger Andreas Dilger added a comment - Is there already a patch for this somewhere? I thought there was.

          People

            jamesanunez James Nunez (Inactive)
            jamesanunez James Nunez (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: