Skip to content

test_in_tail: fix flaky tests for file rotation - #4710

Merged
daipom merged 2 commits into
fluent:masterfrom
Watson1978:fix-test_in_tail
Nov 20, 2024
Merged

test_in_tail: fix flaky tests for file rotation#4710
daipom merged 2 commits into
fluent:masterfrom
Watson1978:fix-test_in_tail

Conversation

@Watson1978

@Watson1978 Watson1978 commented Nov 19, 2024

Copy link
Copy Markdown
Contributor

Which issue(s) this PR fixes:
Fixes #4707

What this PR does / why we need it:
This PR will fix unstable tests.

Docs Changes:

Release Note:

@daipom daipom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!! Great!!

Comment thread test/plugin/test_in_tail.rb Outdated
Comment thread test/plugin/test_in_tail.rb Outdated
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978
Watson1978 marked this pull request as ready for review November 20, 2024 00:30
@Watson1978
Watson1978 requested a review from daipom November 20, 2024 00:31

@daipom daipom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! Thanks!

@daipom
daipom merged commit 2b33a69 into fluent:master Nov 20, 2024
@Watson1978
Watson1978 deleted the fix-test_in_tail branch November 20, 2024 02:14
@daipom daipom added the backport to v1.16 We will backport this fix to the LTS branch label Nov 29, 2024
@daipom daipom added this to the v1.18.0 milestone Jan 28, 2025
kenhys pushed a commit that referenced this pull request Jan 29, 2025
* test_in_tail: fix flaky tests for file rotation

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>

* test_in_tail: remove unnecessary inode assertions

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>

---------

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
daipom pushed a commit that referenced this pull request Jan 29, 2025
… (#4795)

**Which issue(s) this PR fixes**: 

Backport #4710 

**What this PR does / why we need it**: 

This PR will fix unstable tests.

* test_in_tail: fix flaky tests for file rotation
* test_in_tail: remove unnecessary inode assertions

**Docs Changes**:

**Release Note**:

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
Co-authored-by: Watson <fujita@clear-code.com>
@kenhys kenhys added the backported "backport to LTS" is done label Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to v1.16 We will backport this fix to the LTS branch backported "backport to LTS" is done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: Flaky "test_next_rotation_occurs_very_fast_while_old_TW_still_waiting_rotate_wait" test

3 participants