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

lctl llog_print do not print all record

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.16.0, Lustre 2.15.4
    • Lustre 2.15.0, Lustre 2.15.1
    • 3
    • 9223372036854775807

    Description

      If a lot of contiguous records are skipped (>64), llog_print will stop the processing and will not display the last records.

      With the https://review.whamcloud.com/45332 ("LU-15142 lctl: fixes for set_param -P and llog_print"), the following code is not true anymore:

       int jt_llog_print_iter(char *logname, long start, long end,
      ...
                      /* There is no "end of list" marker, record was not modified */
                      if (strcmp(record, logname) == 0)                <------
                              break;                                                 
      

      "record" could be unmodified if all processed records are skipped.

      "llog_print -r" is not affected by this issue.

      Attachments

        Issue Links

          Activity

            [LU-16167] lctl llog_print do not print all record

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51221/
            Subject: LU-16167 obdclass: fix lctl llog_print with skipped records
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set:
            Commit: 73888fd0970dd6e45162b9244edd4573bfe7aae6

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/51221/ Subject: LU-16167 obdclass: fix lctl llog_print with skipped records Project: fs/lustre-release Branch: b2_15 Current Patch Set: Commit: 73888fd0970dd6e45162b9244edd4573bfe7aae6

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51221
            Subject: LU-16167 obdclass: fix lctl llog_print with skipped records
            Project: fs/lustre-release
            Branch: b2_15
            Current Patch Set: 1
            Commit: 9db96d00238d23b7bdbfd020919b87e636b7c532

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/51221 Subject: LU-16167 obdclass: fix lctl llog_print with skipped records Project: fs/lustre-release Branch: b2_15 Current Patch Set: 1 Commit: 9db96d00238d23b7bdbfd020919b87e636b7c532
            pjones Peter Jones added a comment -

            Landed for 2.16

            pjones Peter Jones added a comment - Landed for 2.16

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48586/
            Subject: LU-16167 obdclass: fix lctl llog_print with skipped records
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: c6da54aa7546440339265c644538d3d109e46bde

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48586/ Subject: LU-16167 obdclass: fix lctl llog_print with skipped records Project: fs/lustre-release Branch: master Current Patch Set: Commit: c6da54aa7546440339265c644538d3d109e46bde

            "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/48586
            Subject: LU-16167 obdclass: fix lctl llog_print with skipped records
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 8ef9877a6ec756d0ef1da5a8827d861cac63fd9d

            gerrit Gerrit Updater added a comment - "Etienne AUJAMES <eaujames@ddn.com>" uploaded a new patch: https://review.whamcloud.com/48586 Subject: LU-16167 obdclass: fix lctl llog_print with skipped records Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 8ef9877a6ec756d0ef1da5a8827d861cac63fd9d

            People

              eaujames Etienne Aujames
              eaujames Etienne Aujames
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: