varnishcache / varnish-cache

Varnish Cache source code repository
https://www.varnish-cache.org
Other
3.56k stars 366 forks source link

DONOTMERGE: Investigating archlinux failure on circlecli #4051

Closed dridi closed 4 months ago

dridi commented 5 months ago

Not expecting much from the initial debugging attempt.

dridi commented 5 months ago

For some reason all directories created under _build/sub/ during distcheck have friendly permissions except the offending one:

$ awk '/^d/ {print $1}' /tmp/_build.txt | sort | uniq -c
     90 drwxr-xr-x
      1 dr-x------
$ awk '/^dr-/' /tmp/_build.txt
dr-x------  2 varnish varnish    4096 Feb  6 08:27 varnish-trunk/_build/sub/varnish-trunk/doc/sphinx/dev-guide

Why this directory and not another?

$ awk '/^d/ && $NF ~ "sphinx/" {print $1 " " $NF}' /tmp/_build.txt
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/dev-guide
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/glossary
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/include
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/installation
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/phk
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/reference
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/tutorial
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/users-guide
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/vcl-design-patterns
drwxr-xr-x varnish-trunk/_build/sub/doc/sphinx/whats-new
dr-x------ varnish-trunk/_build/sub/varnish-trunk/doc/sphinx/dev-guide
drwxr-xr-x varnish-trunk/_build/sub/varnish-trunk/doc/sphinx/include

The only other sub-directory that was created under varnish-trunk/_build/sub/varnish-trunk/ is include/ and it got the expected permissions. This directories are copied sequentially in a shell for loop, and dev-guide/ is the first directory in $(EXTRA_DIST). How did include/ show up first? Not via the offending cp -fpR, it is empty. Possibly because of the generated includes.

Is this racing with another target? It shouldn't since distdir-am appears to be called recursively (and not indirectly via the distdir target).

dridi commented 4 months ago

This one appears to be resolved, so either something changed on the archlinux side, or the docker setup part of #4063 made the race go away.