coreutils: tests/tail-2/inotify-dir-recreate.sh fails on overlayfs

OpenSubmitted by Carl Dong.
Details
2 participants
  • Carl Dong
  • Ludovic Courtès
Owner
unassigned
Severity
important
C
C
Carl Dong wrote on 21 Apr 17:35 +0200
(address . bug-guix@gnu.org)
A93F37F7-ADA3-48FF-A1D4-800842F706AB@carldong.me
Hi all,
I’m continuing my testing of the 1.3.0 branch, and I’ve found that coreutil's tests/tail-2/inotify-dir-recreate.sh fails on filesystems where tail detects that it cannot use inotify safely (probably arising out of this check: https://github.com/coreutils/coreutils/blob/34a48bf0f0552aaed21a7dba4a5488946a978317/src/tail.c#L2486-L2491).Example for this: overlayfs used by docker/podman/etc.
The author of tail’s inotify support explains it here: https://github.com/containers/podman/issues/5493#issuecomment-598851397
A quick and easy fix for v1.3.0 may be to just disable tests/tail-2/inotify-dir-recreate.sh :-)
Cheers,Carl Dongcontact@carldong.me"I fight for the users"
C
C
Carl Dong wrote on 22 Apr 03:20 +0200
(address . 47935@debbugs.gnu.org)
C38D9CC8-735D-45F9-83FB-CB7F07F0ACFC@carldong.me
L
L
Ludovic Courtès wrote on 27 Apr 09:53 +0200
control message for bug #47935
(address . control@debbugs.gnu.org)
87czug6u43.fsf@gnu.org
severity 47935 importantquit
L
L
Ludovic Courtès wrote on 3 Jun 21:29 +0200
Re: bug#47935: coreutils: tests/tail-2/inotify-dir-recreate.sh fails on overlayfs
(name . Carl Dong)(address . contact@carldong.me)(address . 47935@debbugs.gnu.org)
87y2bqu4qi.fsf@gnu.org
Hi Carl,
Carl Dong <contact@carldong.me> skribis:
Toggle quote (4 lines)> I’m continuing my testing of the 1.3.0 branch, and I’ve found that coreutil's tests/tail-2/inotify-dir-recreate.sh fails on filesystems where tail detects that it cannot use inotify safely (probably arising out of this check: https://github.com/coreutils/coreutils/blob/34a48bf0f0552aaed21a7dba4a5488946a978317/src/tail.c#L2486-L2491).Example for this: overlayfs used by docker/podman/etc.>> The author of tail’s inotify support explains it here: https://github.com/containers/podman/issues/5493#issuecomment-598851397
[...]
Toggle quote (2 lines)> Upstream bug filed: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=47940
With people more on more interested in finally getting ‘core-updates’merged, now’s a good time to look into this bug. :-)
In the bug report upstream, you hint at a possible fix. Could youprovide a patch for this?
Alternatively, if that turns out to be too tricky, can you think of aworkaround, such as skipping the offending code during tests?
Thanks,Ludo’.
?