8388a64cd1
SZEDER reported that t5411 failed in Travis CI's s390x environment a
couple of times, and could be reproduced with '--stress' test on this
specific environment. The test failure messages might look like this:
+ test_cmp expect actual
--- expect 2021-01-17 21:55:23.430750004 +0000
+++ actual 2021-01-17 21:55:23.430750004 +0000
@@ -1 +1 @@
-<COMMIT-A> refs/heads/main
+<COMMIT-A> refs/heads/maifatal: the remote end hung up unexpectedly
error: last command exited with $?=1
not ok 86 - proc-receive: not support push options (builtin protocol)
The file 'actual' is filtered from the file 'out' which contains result
of 'git show-ref' command. Due to the error messages from other process
is written into the file 'out' accidentally, t5411 failed. SZEDER finds
the root cause of this issue:
- 'git push' is executed with its standard output and error redirected
to the file 'out'.
- 'git push' executes 'git receive-pack' internally, which inherits
the open file descriptors, so its output and error goes into that
same 'out' file.
- 'git push' ends without waiting for the close of 'git-receive-pack'
for some cases, and the file 'out' is reused for test of
'git show-ref' afterwards.
- A mixture of the output of 'git show-ref' abd 'git receive-pack'
leads to this issue.
The first intuitive reaction to resolve this issue is to remove the
file 'out' after use, so that the newly created file 'out' will have a
different file descriptor and will not be overwritten by the
'git receive-pack' process. But Johannes pointed out that removing an
open file is not possible on Windows. So we use different temporary
file names to store the output of 'git push' to solve this issue.
Reported-by: SZEDER Gábor <szeder.dev@gmail.com>
Helped-by: Johannes Sixt <j6t@kdbg.org>
Helped-by: SZEDER Gábor <szeder.dev@gmail.com>
Signed-off-by: Jiang Xin <zhiyou.jx@alibaba-inc.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
92 lines
3.6 KiB
Bash
92 lines
3.6 KiB
Bash
test_expect_success "setup proc-receive hook ($PROTOCOL/porcelain)" '
|
|
write_script "$upstream/hooks/proc-receive" <<-EOF
|
|
printf >&2 "# proc-receive hook\n"
|
|
test-tool proc-receive -v \
|
|
-r "ok refs/for/next/topic2" \
|
|
-r "ng refs/for/next/topic1 fail to call Web API" \
|
|
-r "ok refs/for/main/topic" \
|
|
-r "option refname refs/for/main/topic" \
|
|
-r "option old-oid $A" \
|
|
-r "option new-oid $B"
|
|
EOF
|
|
'
|
|
|
|
# Refs of upstream : main(A)
|
|
# Refs of workbench: main(A) tags/v123
|
|
# git push : (B) bar(A) baz(A) refs/for/next/topic(A) foo(A) refs/for/main/topic(A)
|
|
test_expect_success "proc-receive: report update of mixed refs ($PROTOCOL/porcelain)" '
|
|
test_must_fail git -C workbench push --porcelain origin \
|
|
$B:refs/heads/main \
|
|
HEAD:refs/heads/bar \
|
|
HEAD:refs/heads/baz \
|
|
HEAD:refs/for/next/topic2 \
|
|
HEAD:refs/for/next/topic1 \
|
|
HEAD:refs/heads/foo \
|
|
HEAD:refs/for/main/topic \
|
|
HEAD:refs/for/next/topic3 \
|
|
>out-$test_count 2>&1 &&
|
|
make_user_friendly_and_stable_output <out-$test_count >actual &&
|
|
cat >expect <<-EOF &&
|
|
remote: # pre-receive hook
|
|
remote: pre-receive< <COMMIT-A> <COMMIT-B> refs/heads/main
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/heads/bar
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/heads/baz
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic2
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic1
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/heads/foo
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/for/main/topic
|
|
remote: pre-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic3
|
|
remote: # proc-receive hook
|
|
remote: proc-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic2
|
|
remote: proc-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic1
|
|
remote: proc-receive< <ZERO-OID> <COMMIT-A> refs/for/main/topic
|
|
remote: proc-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic3
|
|
remote: proc-receive> ok refs/for/next/topic2
|
|
remote: proc-receive> ng refs/for/next/topic1 fail to call Web API
|
|
remote: proc-receive> ok refs/for/main/topic
|
|
remote: proc-receive> option refname refs/for/main/topic
|
|
remote: proc-receive> option old-oid <COMMIT-A>
|
|
remote: proc-receive> option new-oid <COMMIT-B>
|
|
remote: # post-receive hook
|
|
remote: post-receive< <COMMIT-A> <COMMIT-B> refs/heads/main
|
|
remote: post-receive< <ZERO-OID> <COMMIT-A> refs/heads/bar
|
|
remote: post-receive< <ZERO-OID> <COMMIT-A> refs/heads/baz
|
|
remote: post-receive< <ZERO-OID> <COMMIT-A> refs/for/next/topic2
|
|
remote: post-receive< <ZERO-OID> <COMMIT-A> refs/heads/foo
|
|
remote: post-receive< <COMMIT-A> <COMMIT-B> refs/for/main/topic
|
|
To <URL/of/upstream.git>
|
|
<COMMIT-B>:refs/heads/main <OID-A>..<OID-B>
|
|
* HEAD:refs/heads/bar [new branch]
|
|
* HEAD:refs/heads/baz [new branch]
|
|
* HEAD:refs/for/next/topic2 [new reference]
|
|
* HEAD:refs/heads/foo [new branch]
|
|
HEAD:refs/for/main/topic <OID-A>..<OID-B>
|
|
! HEAD:refs/for/next/topic1 [remote rejected] (fail to call Web API)
|
|
! HEAD:refs/for/next/topic3 [remote rejected] (proc-receive failed to report status)
|
|
Done
|
|
EOF
|
|
test_cmp expect actual &&
|
|
git -C "$upstream" show-ref >out &&
|
|
make_user_friendly_and_stable_output <out >actual &&
|
|
cat >expect <<-EOF &&
|
|
<COMMIT-A> refs/heads/bar
|
|
<COMMIT-A> refs/heads/baz
|
|
<COMMIT-A> refs/heads/foo
|
|
<COMMIT-B> refs/heads/main
|
|
EOF
|
|
test_cmp expect actual
|
|
'
|
|
|
|
# Refs of upstream : main(B) foo(A) bar(A)) baz(A)
|
|
# Refs of workbench: main(A) tags/v123
|
|
test_expect_success "cleanup ($PROTOCOL/porcelain)" '
|
|
(
|
|
cd "$upstream" &&
|
|
git update-ref refs/heads/main $A &&
|
|
git update-ref -d refs/heads/foo &&
|
|
git update-ref -d refs/heads/bar &&
|
|
git update-ref -d refs/heads/baz
|
|
)
|
|
|
|
'
|