Skip to content

test_tail: increase delay for test_follow_name_move_create2#3606

Merged
sylvestre merged 1 commit intouutils:mainfrom
jhscheer:tail_notify
Jun 10, 2022
Merged

test_tail: increase delay for test_follow_name_move_create2#3606
sylvestre merged 1 commit intouutils:mainfrom
jhscheer:tail_notify

Conversation

@jhscheer
Copy link
Copy Markdown
Contributor

@jhscheer jhscheer commented Jun 9, 2022

Increase delay for this test to be more resillient against
load-jitter on the CI test VMs.

I hope this helps to solve #2695 (comment)

Increase delay for this test to be more resillient against
load-jitter on the CI test VMs.
@sylvestre sylvestre merged commit d1f7f51 into uutils:main Jun 10, 2022
@sylvestre
Copy link
Copy Markdown
Contributor

Happened again here:

https://github.com/uutils/coreutils/runs/6838926336?check_suite_focus=true

---- test_tail::test_follow_name_move_create2 stdout ----
current_directory_resolved: 
touch: /tmp/.tmp7VSGCz/1
touch: /tmp/.tmp7VSGCz/2
touch: /tmp/.tmp7VSGCz/3
touch: /tmp/.tmp7VSGCz/4
touch: /tmp/.tmp7VSGCz/5
touch: /tmp/.tmp7VSGCz/6
touch: /tmp/.tmp7VSGCz/7
touch: /tmp/.tmp7VSGCz/8
touch: /tmp/.tmp7VSGCz/9
run: /home/runner/work/coreutils/coreutils/target/debug/coreutils tail -s.1 --max-unchanged-stats=1 -q -F 1 2 3 4 5 6 7 8 9
write(truncate): /tmp/.tmp7VSGCz/9
rename: "/tmp/.tmp7VSGCz/1" "/tmp/.tmp7VSGCz/f"
write(truncate): /tmp/.tmp7VSGCz/1
remove: "/tmp/.tmp7VSGCz/f"
run: /home/runner/work/coreutils/coreutils/target/debug/coreutils tail -s.1 --max-unchanged-stats=1 -q -F 1 2 3 4 5 6 7 8 9 ---disable-inotify
write(truncate): /tmp/.tmp7VSGCz/9
rename: "/tmp/.tmp7VSGCz/1" "/tmp/.tmp7VSGCz/f"
write(truncate): /tmp/.tmp7VSGCz/1
thread 'test_tail::test_follow_name_move_create2' panicked at 'assertion failed: `(left == right)`
  left: `"tail: '1' has become inaccessible: No such file or directory\ntail: 1: file truncated\n"`,
 right: `"tail: '1' has become inaccessible: No such file or directory\ntail: '1' has appeared;  following new file\n"`', /home/runner/work/coreutils/coreutils/tests/by-util/test_tail.rs:1898:9

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