Page MenuHomeFreeBSD
Differential D29267

PR 253868: Periodic issues
ClosedPublic
Actions

Authored by ceri on Mar 15 2021, 9:48 AM.
Tags
None
Referenced Files
F132384414: D29267.diff
Thu, Oct 16, 10:53 AM
Unknown Object (File)
Tue, Oct 14, 1:17 PM
Unknown Object (File)
Tue, Oct 7, 5:45 AM
Unknown Object (File)
Mon, Oct 6, 6:51 PM
Unknown Object (File)
Sep 8 2025, 4:24 AM
Unknown Object (File)
Aug 25 2025, 7:31 AM
Unknown Object (File)
Aug 15 2025, 11:18 PM
Unknown Object (File)
Jul 21 2025, 6:12 PM
View All 86 Files
Subscribers
allanjude

Details

Reviewers
allanjude
Group Reviewers
Contributor Reviews (src)
Commits
rG774bb1c256fb: periodic: add support for .xz and .zcat compressed logs
Summary

Patch to address issues described.

  1. /etc/periodic has special cases for zcat and bzcat, but not for zstd and xz which newsyslog also supports.
  1. newsyslog.conf manpage does not have a xref for zstd(1).
  1. usr.sbin/periodic/etc/monthly/200.accounting does not use temporary files appropriately.
  1. periodic.conf(5) refers to "daily accounting" and "monthly accounting" withouth being clear that "daily" means process accounting and "monthly" is login accounting.
Test Plan

Has been running on my system for 18 days now.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ceri requested review of this revision.Mar 15 2021, 9:48 AM
ceri created this revision.
lwhsu added a reviewer: Contributor Reviews (src).Mar 16 2021, 4:34 AM
allanjude accepted this revision as: allanjude.Jun 2 2021, 12:09 PM
allanjude added a subscriber: allanjude.

Reviewed By: allanjude

This revision is now accepted and ready to land.Jun 2 2021, 12:09 PM
Closed by commit rG774bb1c256fb: periodic: add support for .xz and .zcat compressed logs (authored by ceri). · Explain WhyJun 2 2021, 4:34 PM
This revision was automatically updated to reflect the committed changes.
ceri added a commit: rG774bb1c256fb: periodic: add support for .xz and .zcat compressed logs.

Revision Contents
Changeset List

PathSize
share/
man/
man5/
2 lines
usr.sbin/
newsyslog/
1 line
periodic/
etc/
daily/
6 lines
monthly/
30 lines
security/
2 lines
2 lines

Diff 90329

View Options

share/man/man5/periodic.conf.5

Loading...
View Options

usr.sbin/newsyslog/newsyslog.conf.5

Loading...
View Options

usr.sbin/periodic/etc/daily/460.status-mail-rejects

Loading...
View Options

usr.sbin/periodic/etc/monthly/200.accounting

Loading...
View Options

usr.sbin/periodic/etc/security/800.loginfail

Loading...
View Options

usr.sbin/periodic/etc/security/900.tcpwrap

Loading...