From: Josh Steadmon <steadmon@google.com>
To: git@vger.kernel.org
Cc: bcmills@google.com, peff@peff.net
Subject: `git fetch` with protocol.version=1 misses tags that point to the fetched history
Date: Tue, 23 Jan 2024 13:08:01 -0800 [thread overview]
Message-ID: <ZbAqsf-2DWaXbN7K@google.com> (raw)
At $DAYJOB, we got a bug report that Git 2.21.0 breaks Go's CI due to
not fetching all tags in the history. The bug reporter (Bryan, CCed) was
kind enough to bisect this failure down to 61c7711cfe (sha1-file: use
loose object cache for quick existence check, 2018-11-12). This was only
recently discovered because Go's CI was using Git v2.17.6.
More details can be found in the original bug report [1] and Go's issue
for the CI breakage [2].
[1] https://git.g-issues.gerritcodereview.com/issues/320678525
[2] https://github.com/golang/go/issues/56881
A log of the failing Go test follows:
vcs-test.golang.org rerouted to http://127.0.0.1:36865
https://vcs-test.golang.org rerouted to https://127.0.0.1:43597
=== RUN TestStat
=== PAUSE TestStat
=== CONT TestStat
=== RUN TestStat/gitrepo1/v1.2.4-annotated
=== PAUSE TestStat/gitrepo1/v1.2.4-annotated
=== CONT TestStat/gitrepo1/v1.2.4-annotated
git_test.go:166: mkdir -p /tmp/gitrepo-test-767558581/modcache/cache/vcs # git3 http://127.0.0.1:36865/git/gitrepo1
git_test.go:166: # lock /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0.lock
git_test.go:166: mkdir -p /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0 # git3 http://127.0.0.1:36865/git/gitrepo1
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git init --bare
git_test.go:166: 0.011s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git init --bare
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git remote add origin -- http://127.0.0.1:36865/git/gitrepo1
git_test.go:166: 0.008s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git remote add origin -- http://127.0.0.1:36865/git/gitrepo1
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git tag -l
git_test.go:166: 0.008s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git tag -l
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git ls-remote -q origin
2024/01/17 14:11:49 serving /git/gitrepo1/info/refs?service=git-upload-pack
2024/01/17 14:11:49 gitrepo1.txt:
> handle git
> env GIT_AUTHOR_NAME='Russ Cox'
> env GIT_AUTHOR_EMAIL='rsc@golang.org'
> env GIT_COMMITTER_NAME=$GIT_AUTHOR_NAME
> env GIT_COMMITTER_EMAIL=$GIT_AUTHOR_EMAIL
> git init
[stdout]
Initialized empty Git repository in /tmp/vcstest1128851619/git/gitrepo1/.git/
> at 2018-04-17T15:43:22-04:00
> unquote ''
> cp stdout README
> git add README
> git commit -a -m 'empty README'
[stdout]
[main (root-commit) ede458d] empty README
1 file changed, 0 insertions(+), 0 deletions(-)
create mode 100644 README
> git branch -m master
> git tag v1.2.3
> at 2018-04-17T15:45:48-04:00
> git branch v2
> git checkout v2
[stderr]
Switched to branch 'v2'
> echo 'v2'
[stdout]
v2
> cp stdout v2
> git add v2
> git commit -a -m 'v2'
[stdout]
[v2 76a00fb] v2
1 file changed, 1 insertion(+)
create mode 100644 v2
> git tag v2.3
> git tag v2.0.1
> git branch v2.3.4
> at 2018-04-17T16:00:19-04:00
> echo 'intermediate'
[stdout]
intermediate
> cp stdout foo.txt
> git add foo.txt
> git commit -a -m 'intermediate'
[stdout]
[v2 97f6aa5] intermediate
1 file changed, 1 insertion(+)
create mode 100644 foo.txt
> at 2018-04-17T16:00:32-04:00
> echo 'another'
[stdout]
another
> cp stdout another.txt
> git add another.txt
> git commit -a -m 'another'
[stdout]
[v2 9d02800] another
1 file changed, 1 insertion(+)
create mode 100644 another.txt
> git tag v2.0.2
> at 2018-04-17T16:16:52-04:00
> git checkout master
[stderr]
Switched to branch 'master'
> git branch v3
> git checkout v3
[stderr]
Switched to branch 'v3'
> mkdir v3/sub/dir
> echo 'v3/sub/dir/file'
[stdout]
v3/sub/dir/file
> cp stdout v3/sub/dir/file.txt
> git add v3
> git commit -a -m 'add v3/sub/dir/file.txt'
[stdout]
[v3 a8205f8] add v3/sub/dir/file.txt
1 file changed, 1 insertion(+)
create mode 100644 v3/sub/dir/file.txt
> at 2018-04-17T22:23:00-04:00
> git checkout master
[stderr]
Switched to branch 'master'
> git tag -a v1.2.4-annotated -m 'v1.2.4-annotated'
> git show-ref --tags --heads
[stdout]
ede458df7cd0fdca520df19a33158086a8a68e81 refs/heads/master
9d02800338b8a55be062c838d1f02e0c5780b9eb refs/heads/v2
76a00fb249b7f93091bc2c89a789dab1fc1bc26f refs/heads/v2.3.4
a8205f853c297ad2c3c502ba9a355b35b7dd3ca5 refs/heads/v3
ede458df7cd0fdca520df19a33158086a8a68e81 refs/tags/v1.2.3
b004e48a345a86ed7a2fb7debfa7e0b2f9b0dd91 refs/tags/v1.2.4-annotated
76a00fb249b7f93091bc2c89a789dab1fc1bc26f refs/tags/v2.0.1
9d02800338b8a55be062c838d1f02e0c5780b9eb refs/tags/v2.0.2
76a00fb249b7f93091bc2c89a789dab1fc1bc26f refs/tags/v2.3
> cmp stdout .git-refs
git_test.go:166: 0.194s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git ls-remote -q origin
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git -c log.showsignature=false log --no-decorate -n1 '--format=format:%H %ct %D' ede458df7cd0fdca520df19a33158086a8a68e81 --
git_test.go:166: 0.006s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git -c log.showsignature=false log --no-decorate -n1 '--format=format:%H %ct %D' ede458df7cd0fdca520df19a33158086a8a68e81 --
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git fetch -f --depth=1 origin refs/tags/v1.2.4-annotated:refs/tags/v1.2.4-annotated
2024/01/17 14:11:49 serving /git/gitrepo1/info/refs?service=git-upload-pack
2024/01/17 14:11:49 serving /git/gitrepo1/git-upload-pack
2024/01/17 14:11:49 serving /git/gitrepo1/git-upload-pack
git_test.go:166: 0.113s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git fetch -f --depth=1 origin refs/tags/v1.2.4-annotated:refs/tags/v1.2.4-annotated
git_test.go:166: cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git -c log.showsignature=false log --no-decorate -n1 '--format=format:%H %ct %D' refs/tags/v1.2.4-annotated --
git_test.go:166: 0.007s # cd /tmp/gitrepo-test-767558581/modcache/cache/vcs/72d702b2b3eb7cb41c516773b2d675c9ee1480e2be1059cbf3bd15f9cfac2bf0; git -c log.showsignature=false log --no-decorate -n1 '--format=format:%H %ct %D' refs/tags/v1.2.4-annotated --
git_test.go:661: Stat: incorrect info
have {Origin:<nil> Name:ede458df7cd0fdca520df19a33158086a8a68e81 Short:ede458df7cd0 Version:v1.2.4-annotated Time:2018-04-17 19:43:22 +0000 UTC Tags:[v1.2.4-annotated]}
want {Origin:<nil> Name:ede458df7cd0fdca520df19a33158086a8a68e81 Short:ede458df7cd0 Version:v1.2.4-annotated Time:2018-04-17 19:43:22 +0000 UTC Tags:[v1.2.3 v1.2.4-annotated]}
--- FAIL: TestStat (0.00s)
--- FAIL: TestStat/gitrepo1/v1.2.4-annotated (0.35s)
FAIL
FAIL cmd/go/internal/modfetch/codehost 0.398s
FAIL
next reply other threads:[~2024-01-23 21:08 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-23 21:08 Josh Steadmon [this message]
2024-01-24 1:00 ` [PATCH] transport-helper: re-examine object dir after fetching Jeff King
2024-01-24 19:31 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZbAqsf-2DWaXbN7K@google.com \
--to=steadmon@google.com \
--cc=bcmills@google.com \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).