Univention Bugzilla – Bug 46279
[4.3] systemd-journald: Please introduce UCR variable for SplitMode
Last modified: 2018-05-02 13:31:20 CEST
Hi We've had a customer whose UCS server with Samba and Proxy had log growth rate that required some 'taming' and we opted to modify SplitMode from 'uid' to 'login', since 'man systemd.conf': SplitMode= Controls whether to split up journal files per user [...] If "login", actually logged-in users will get each their own journal files, but users without login session and system users will log into the system journal. However as of now, there is no such UCR variable available we could use for tuning when needed. We'd (that is, the customer and I) appreciate if you'd introduce systemd/journald/SplitMode as UCR variable. What do you think abou this since there are already other journald-Related options present in UCS 4.2 Best regards Mathieu Simon, Adfinis SyGroup AG
Hi The diff would be pretty easy and has been quickly validated against UCS 4.2-3 --- journald.conf.orig 2018-03-26 18:21:09.328186419 +0200 +++ journald.conf 2018-03-26 18:19:43.700183129 +0200 @@ -17,8 +17,13 @@ print "Storage=%s" % (value, ) @!@#Compress=yes #Seal=yes -#SplitMode=uid -#SyncIntervalSec=5m +@!@ +value = configRegistry.get('systemd/journald/SplitMode') +if value is None: + print "SplitMode=uid" +else: + print "SplitMode=%s" % (value, ) +@!@#SyncIntervalSec=5m #RateLimitInterval=30s #RateLimitBurst=1000 @!@ Yes, the descriptions in (likely?) /etc/univention/templates/info/univention-base-files.info ist still missing. Looking forward to hearing from you. If you prefer a pull request on github, please let me know. -- Mathieu
Hmm, I saw you have moved the source to Github, nifty. So let's try how this goes: https://github.com/univention/univention-corporate-server/pull/5/files
Thank you for the PR! [4.3-0] d25168b4d5 Bug #46279: allow to set journald SplitMode by UCRV [4.3-0] acf792c193 Bug #46279: changelog [4.3-0] 4a6942a049 Bug #46279: advisory univention-base-files (7.0.0-11) Changes to UCS 4.2-3-errata will be tracked through Bug #46750.
I've fixed a trivial typo in the variable description, but I don't think we need to rebuild the package for that (commit a22e7a0e17). Verified: * Code review: Ok * Functional test: Ok * Advisory: Ok
c103fa8483 | Fix trivial typo
<http://errata.software-univention.de/ucs/4.3/23.html>