Skip to content

Bump the default frequency of the logrotate cron job#247

Merged
brianhlin merged 1 commit intoopensciencegrid:mainfrom
matyasselmeci:pr/bump-logrotate
May 16, 2025
Merged

Bump the default frequency of the logrotate cron job#247
brianhlin merged 1 commit intoopensciencegrid:mainfrom
matyasselmeci:pr/bump-logrotate

Conversation

@matyasselmeci
Copy link
Contributor

Since log rotation is based on date or size, this doesn't cause the logs to be rotated more frequently, it just causes logrotate to check for the rotation condition more frequently, allowing it to catch sudden growths in size. logrotate itself has locking so this should be safe.

Since log rotation is based on date or size, this doesn't cause the logs
to be rotated more frequently, it just causes logrotate to check for the
rotation condition more frequently, allowing it to catch sudden growths
in size. logrotate itself has locking so this should be safe.
Copy link
Member

@brianhlin brianhlin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine by me

@brianhlin brianhlin merged commit 8af58d5 into opensciencegrid:main May 16, 2025
4 checks passed
@matyasselmeci matyasselmeci deleted the pr/bump-logrotate branch May 16, 2025 14:41
@matyasselmeci matyasselmeci restored the pr/bump-logrotate branch June 13, 2025 22:19
@matyasselmeci matyasselmeci deleted the pr/bump-logrotate branch June 13, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants