[LU-12871] enable changelog garbage collection by default Created: 17/Oct/19  Updated: 21/Nov/23  Resolved: 28/Jan/20

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.13.0, Lustre 2.12.3
Fix Version/s: Lustre 2.14.0

Type: Improvement Priority: Minor
Reporter: Andreas Dilger Assignee: Andreas Dilger
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
is related to LU-10680 MDT becoming unresponsive in 2.10.3 Resolved
is related to LU-7340 ChangeLogs catalog full condition sho... Resolved
is related to LU-12865 sanity test 160f fails with ‘mds1: Us... Resolved
is related to LU-14699 changelog garbage collection is too lax Resolved
is related to LU-17290 Don't deregister idle changelog consu... Open
Rank (Obsolete): 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.

 Comments   
Comment by Andreas Dilger [ 17/Oct/19 ]

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

Comment by Andreas Dilger [ 17/Oct/19 ]

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.

Comment by Bruno Faccini (Inactive) [ 17/Oct/19 ]

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...

Comment by Gerrit Updater [ 28/Jan/20 ]

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

Comment by Peter Jones [ 28/Jan/20 ]

Landed for 2.14

Generated at Sat Feb 10 02:56:22 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.