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

enable changelog garbage collection by default

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0
    • Lustre 2.13.0, Lustre 2.12.3
    • None
    • 9223372036854775807

    Description

      This feature was disabled by default in commit v2_10_56_0-2-g3442db6fa (2.11.0) and was fixed in commit v2_11_52_0-59-g31fef6845e (2.12), but was not re-enabled again by default.

      This is a very useful functionality as sites often register Changelog users but forget to remove them, causing problems with the MDT running out of space or the Changelog becoming full. Enable it again by default.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-12865 sanity test 160f fails with ‘mds1: User cl6 not registered’

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-14699 changelog garbage collection is too lax

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-17290 Don't deregister idle changelog consumers

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-10680 MDT becoming unresponsive in 2.10.3

          • Major - Major loss of function.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-7340 ChangeLogs catalog full condition should be handled more gracefully

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

          Activity

            [LU-12871] enable changelog garbage collection by default
            pjones Peter Jones added a comment -

            Landed for 2.14

            pjones Peter Jones added a comment - Landed for 2.14
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36467/
            Subject: LU-12871 mdd: enable Changelog garbage collection
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 1cc689723685a564c4d3f45ba44348e2e43a8442

            gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36467/ Subject: LU-12871 mdd: enable Changelog garbage collection Project: fs/lustre-release Branch: master Current Patch Set: Commit: 1cc689723685a564c4d3f45ba44348e2e43a8442
            bruno Bruno Faccini (Inactive) added a comment -

            Well, as my first patch that had landed for this feature (for LU-7340) had to be reverted by John because of MDT hangs it generated (in LU-10680), seems that at the time I finally pushed the final fix to finalize the feature I was reluctant to enable it by default, this clearly appears in my Commit-message for the patch ("Also, enable Changelog GC, as it is no longer the default, in sanity/test_160f sub-test ...").
            And sorry, but I should have made it more clear in LU-10680 at this time...

            bruno Bruno Faccini (Inactive) added a comment - Well, as my first patch that had landed for this feature (for LU-7340 ) had to be reverted by John because of MDT hangs it generated (in LU-10680 ), seems that at the time I finally pushed the final fix to finalize the feature I was reluctant to enable it by default, this clearly appears in my Commit-message for the patch ("Also, enable Changelog GC, as it is no longer the default, in sanity/test_160f sub-test ..."). And sorry, but I should have made it more clear in LU-10680 at this time...
            adilger Andreas Dilger added a comment -

            Bruno, do you recall any reason why this feature was left disabled? I recall we disabled it right before the 2.11 release, but then a patch was landed for 2.12 that fixed the feature and added the test, but it didn't enable the feature again.

            adilger Andreas Dilger added a comment - Bruno, do you recall any reason why this feature was left disabled? I recall we disabled it right before the 2.11 release, but then a patch was landed for 2.12 that fixed the feature and added the test, but it didn't enable the feature again.
            adilger Andreas Dilger added a comment -

            Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36467
            Subject: LU-12871 mdd: enable Changelog garbage collection
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: b8c02cdf67575162778ae2370f1195571bba4fe4

            adilger Andreas Dilger added a comment - Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/36467 Subject: LU-12871 mdd: enable Changelog garbage collection Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: b8c02cdf67575162778ae2370f1195571bba4fe4

            People

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

              Dates

                Created:
                Updated:
                Resolved: