Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cmd/go/internal/vcweb/vcstest: TestScripts failures #56583

Closed
gopherbot opened this issue Nov 4, 2022 · 3 comments
Closed

cmd/go/internal/vcweb/vcstest: TestScripts failures #56583

gopherbot opened this issue Nov 4, 2022 · 3 comments
Labels
FrozenDueToAge GoCommand cmd/go NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Darwin Testing An issue that has been verified to require only test changes, not just a test failure. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone
Backlog

Comments

@gopherbot
Copy link
Contributor

gopherbot commented Nov 4, 2022 •
edited by bcmills
Loading 

#!watchflakes
post <- pkg == "cmd/go/internal/vcweb/vcstest" && test == "TestScripts" && date < "2023-07-01"

Issue created automatically to collect these failures.

Example (log):

2022/10/31 16:09:43 serving /
--- FAIL: TestScripts (0.16s)
    --- FAIL: TestScripts/git/querytest.txt (2.13s)
        vcstest_test.go:136: 2022/10/31 16:09:43 querytest.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
...
            +e0cf3de987e660c21b6950e85b317ce5f7fbb9d9 refs/tags/v1.9.10-pre1
            +42abcb6df8eee6983aeca9a307c28ea40530aceb refs/tags/v1.9.10-pre2+metadata
            +5ba9a4ea62136ae86213feba68bc73858f55b7e1 refs/tags/v1.9.9
             9763aa065ae27c6cacec5ca8b6dfa43a1b31dea0 refs/tags/v2.0.0
             23c28cb696ff40a2839ce406f2c173aa6c3cdda6 refs/tags/v2.1.0
             1828ee9f8074075675013e4d488d5d49ddc1b502 refs/tags/v2.2.0



        vcstest_test.go:142: querytest.txt:243: cmp stdout .git-refs: stdout and .git-refs differ

— watchflakes
@gopherbot gopherbot added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Nov 4, 2022
@gopherbot
Copy link
Contributor Author

Found new dashboard test flakes for:

#!watchflakes
post <- pkg == "cmd/go/internal/vcweb/vcstest" && test == "TestScripts"
2022-10-31 22:19 darwin-amd64-11_0 go@c81c0279 cmd/go/internal/vcweb/vcstest.TestScripts (log) 
2022/10/31 16:09:43 serving /
--- FAIL: TestScripts (0.16s)
    --- FAIL: TestScripts/git/querytest.txt (2.13s)
        vcstest_test.go:136: 2022/10/31 16:09:43 querytest.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
...
            +e0cf3de987e660c21b6950e85b317ce5f7fbb9d9 refs/tags/v1.9.10-pre1
            +42abcb6df8eee6983aeca9a307c28ea40530aceb refs/tags/v1.9.10-pre2+metadata
            +5ba9a4ea62136ae86213feba68bc73858f55b7e1 refs/tags/v1.9.9
             9763aa065ae27c6cacec5ca8b6dfa43a1b31dea0 refs/tags/v2.0.0
             23c28cb696ff40a2839ce406f2c173aa6c3cdda6 refs/tags/v2.1.0
             1828ee9f8074075675013e4d488d5d49ddc1b502 refs/tags/v2.2.0



        vcstest_test.go:142: querytest.txt:243: cmp stdout .git-refs: stdout and .git-refs differ

— watchflakes

@bcmills
Copy link
Contributor

bcmills commented Nov 4, 2022 •
edited
Loading

This looks to me like a platform filesystem bug. git produced the wrong commit hash for files written by the test.

Leaving in the backlog pending more occurrences.

@bcmills bcmills added this to the Backlog milestone Nov 4, 2022
@bcmills bcmills added OS-Darwin GoCommand cmd/go Testing An issue that has been verified to require only test changes, not just a test failure. labels Nov 4, 2022
@bcmills bcmills added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 15, 2023
@gopherbot
Copy link
Contributor Author

Timed out in state WaitingForInfo. Closing.

(I am just a bot, though. Please speak up if this is a mistake or you have the requested information.)

@gopherbot gopherbot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 15, 2023
@golang golang locked and limited conversation to collaborators Jul 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge GoCommand cmd/go NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Darwin Testing An issue that has been verified to require only test changes, not just a test failure. WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
Test Flakes
Archived in project
Milestone
Backlog
Development

No branches or pull requests
2 participants
@bcmills @gopherbot