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

Draft: test: in_tail: Add tests for rotation with follow_inodes #4207

Conversation

daipom
Copy link
Contributor

@daipom daipom commented Jun 20, 2023

Which issue(s) this PR fixes:
None.
Related to #3614.

What this PR does / why we need it:
This PR adds tests to reproduce the problems with follow_inodes, especially talked in #3614.
At this point, several fail.
We need to fix in_tail so that these tests succeed.

Perhaps, we don't need to merge this PR.
I created this PR to share these tests and discuss them.
We can add these tests to the future PR that actually fixes in_tail.

Docs Changes:
No need.

Release Note:
No need.

Many problems related to rotation have been reported.
These tests are to reproduce the problems with follow_inodes,
especially talked in fluent#3614.
At this point, several fail.
We need to fix in_tail so that these tests succeed.

Signed-off-by: Daijiro Fukuda <[email protected]>
@daipom
Copy link
Contributor Author

daipom commented Jun 27, 2023

These tests are added in the following PR:

@daipom daipom closed this Jun 27, 2023
@daipom daipom deleted the test-add-in_tail-rotation-with-follow_inodes branch June 27, 2023 02:06
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.

Fluentd in_tail "unreadable" file causes "following tail of <file>" to stop and no logs pushed
1 participant