Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correct rotation bug in logrotate #1024

Merged
merged 2 commits into from
Oct 3, 2024
Merged

Conversation

blt
Copy link
Collaborator

@blt blt commented Oct 2, 2024

What does this PR do?

This commit removes a bug in the logrotate rotation mechanism where
renames would cause the child generator crash if more than one historical
copy of a log file was kept on disk.

@blt blt marked this pull request as ready for review October 2, 2024 21:02
@blt blt requested a review from a team as a code owner October 2, 2024 21:02
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from 10fe605 to d57289e Compare October 2, 2024 21:03
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from 197c443 to 75c6706 Compare October 2, 2024 21:03
@blt blt removed the no-changelog label Oct 2, 2024
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from d57289e to dd09a2f Compare October 3, 2024 00:08
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from f0d2307 to f442907 Compare October 3, 2024 00:08
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from dd09a2f to fb34d2e Compare October 3, 2024 00:40
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from f442907 to d4049ef Compare October 3, 2024 00:40
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from fb34d2e to 687ad7b Compare October 3, 2024 01:14
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from d4049ef to d48903d Compare October 3, 2024 01:15
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from 687ad7b to f9255a3 Compare October 3, 2024 01:50
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from d48903d to 6dd8100 Compare October 3, 2024 01:51
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from f9255a3 to a1669f9 Compare October 3, 2024 02:26
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from 6dd8100 to bda20aa Compare October 3, 2024 02:27
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from a1669f9 to 5deb7c7 Compare October 3, 2024 03:01
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from bda20aa to 987147c Compare October 3, 2024 03:02
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from 5deb7c7 to c3f1183 Compare October 3, 2024 03:34
@blt blt force-pushed the blt/make_io_errors_paintful_explicit_in_logrotate_generator branch from c3f1183 to d2f74d9 Compare October 3, 2024 03:35
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from 987147c to 4054f9f Compare October 3, 2024 03:35
@blt blt changed the base branch from blt/make_io_errors_paintful_explicit_in_logrotate_generator to graphite-base/1024 October 3, 2024 04:11
This commit removes a bug in the logrotate rotation mechanism where
renames would cause the child generator crash if more than one historical
copy of a log file was kept on disk.

Signed-off-by: Brian L. Troutwine <[email protected]>
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from 4054f9f to f3562af Compare October 3, 2024 04:11
@blt blt changed the base branch from graphite-base/1024 to main October 3, 2024 04:12
Signed-off-by: Brian L. Troutwine <[email protected]>
@blt blt force-pushed the blt/correct_rotation_bug_in_logrotate branch from f3562af to 430eb7e Compare October 3, 2024 04:12
@blt blt merged commit 7740f86 into main Oct 3, 2024
14 checks passed
Copy link
Collaborator Author

blt commented Oct 3, 2024

Merge activity

  • Oct 3, 12:44 AM EDT: @blt merged this pull request with Graphite.

@blt blt deleted the blt/correct_rotation_bug_in_logrotate branch October 3, 2024 04:44
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