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

ChangeLogs catalog full condition should be handled more gracefully

Details

    • Improvement
    • Resolution: Fixed
    • Critical
    • Lustre 2.11.0, Lustre 2.10.3
    • Lustre 2.8.0, Lustre 2.5.4
    • None
    • 9223372036854775807

    Description

      Presently when a LLOG Catalog wraps and its latest assigned index collides with its oldest and still in use index, ENOSPC is returned and the caller just ignore the fact that LLOG record could not be written.

      For ChangeLogs specific usage, some actions could be attempted to recover space/records, some ideas have already been detailed in LU-6556, but it seems better to address them in this separate ticket.

      Input from Andreas :
      I think the other thing that is needed here is to automatically unregister ChangeLog watcher(s) if the changelog is full or the MDS runs out of space (by default), or block all MDS operations until the ChangeLog can be written (if /proc tunable is set to make ChangeLog updates mandatory). It should unregister starting with the oldest watcher on the assumption that the older watcher was forgotten and newer ones are still running, and that this will release the most space. The unregistration should cancel records up to the next watcher, or all remaining records if no other watchers are left.

      Input from Robert :
      I suggest going a step further and proactively remove stale watchers after a configurable period or when hitting a max watermark to try o avoid running out of space. Also, being unregistered is a reasonable notification to the application that they've lost their changelog feed and need to resync.

      Attachments

        Issue Links

          Activity

            [LU-7340] ChangeLogs catalog full condition should be handled more gracefully

            John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30573/
            Subject: LU-7340 mdd: changelogs garbage collection
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set:
            Commit: b87511a0578a03447c51a8495966d60c90fcee61

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) merged in patch https://review.whamcloud.com/30573/ Subject: LU-7340 mdd: changelogs garbage collection Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: b87511a0578a03447c51a8495966d60c90fcee61

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30573
            Subject: LU-7340 mdd: changelogs garbage collection
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 12c8163c17149d2afed78e2ba84da624dd920b34

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/30573 Subject: LU-7340 mdd: changelogs garbage collection Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 12c8163c17149d2afed78e2ba84da624dd920b34
            pjones Peter Jones added a comment -

            Landed for 2.11

            pjones Peter Jones added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/27103/
            Subject: LU-7340 mdd: changelogs garbage collection
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 3442db6faf685fbdbd092bdfdc8d273e4990a141

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/27103/ Subject: LU-7340 mdd: changelogs garbage collection Project: fs/lustre-release Branch: master Current Patch Set: Commit: 3442db6faf685fbdbd092bdfdc8d273e4990a141

            Faccini Bruno (bruno.faccini@intel.com) uploaded a new patch: https://review.whamcloud.com/27103
            Subject: LU-7340 mdd: changelogs garbage collection
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: b7d9cc253398f8f1a07038c351a2ce1a8969d834

            gerrit Gerrit Updater added a comment - Faccini Bruno (bruno.faccini@intel.com) uploaded a new patch: https://review.whamcloud.com/27103 Subject: LU-7340 mdd: changelogs garbage collection Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: b7d9cc253398f8f1a07038c351a2ce1a8969d834

            Sorry to be late on this, but I should be able to provide a fix/solution soon now.

            bfaccini Bruno Faccini (Inactive) added a comment - Sorry to be late on this, but I should be able to provide a fix/solution soon now.

            People

              bfaccini Bruno Faccini (Inactive)
              bfaccini Bruno Faccini (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: