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.

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

Unstable test github.com/pingcap/tidb/plugin.Shutdown #28054

Closed
Tracked by #25899
zhouqiang-cl opened this issue Sep 15, 2021 · 6 comments
Closed
Tracked by #25899

Unstable test github.com/pingcap/tidb/plugin.Shutdown #28054

zhouqiang-cl opened this issue Sep 15, 2021 · 6 comments
Assignees
Labels
component/test severity/major sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@zhouqiang-cl
Copy link
Contributor

Bug Report

[plugin.go:362] ["call OnShutdown for failure"] [plugin=tplugin] [error=EOF] [stack="github.com/pingcap/tidb/plugin.Shutdown\n\t/home/jenkins/agent/workspace/tidb_ghpr_check_2/go/src/github.com/pingcap/tidb/plugin/plugin.go:362\ngithub.com/pingcap/tidb/plugin.TestLoadPluginSkipError\n\t/home/jenkins/agent/workspace/tidb_ghpr_check_2/go/src/github.com/pingcap/tidb/plugin/plugin_test.go:174\ntesting.tRunner\n\t/usr/local/go/src/testing/testing.go:1193"]

Please answer these questions before submitting your issue. Thanks!

1. Minimal reproduce step (Required)

in ci https://ci.pingcap.net/blue/organizations/jenkins/tidb_ghpr_check_2/detail/tidb_ghpr_check_2/33435/pipeline

2. What did you expect to see? (Required)

3. What did you see instead (Required)

4. What is your TiDB version? (Required)

@zimulala
Copy link
Contributor

I can't find specific logs of test failures.
The Call OnShutdown for Failure log is also printed if the test passes.

@zimulala
Copy link
Contributor

@zhouqiang-cl
If you have a similar error, please post the link here.

@aytrack
Copy link
Contributor

aytrack commented Nov 15, 2021

@jebter jebter added the sig/sql-infra SIG: SQL Infra label Dec 7, 2021
@djshow832
Copy link
Contributor

djshow832 commented Dec 7, 2021

According to the logs in https://ci.pingcap.net/blue/organizations/jenkins/tidb_ghpr_coverage/detail/tidb_ghpr_coverage/1191/pipeline, the last running test was TestAuditLog and all the statements (in which the last statement is shutdown) were finished. The case was added on 5th, Aug, and this CI failure begins on 15th, Sep. I doubt there may be a relationship between them but the TestAuditLog was removed and changed to another test case recently (after the last failure).
Let's wait and see if there are more failures.

@bb7133
Copy link
Member

bb7133 commented Dec 9, 2021

Close this issue and we can try to open it, when the fail happens again after #30165.

@github-actions
Copy link

github-actions bot commented Dec 9, 2021

Please check whether the issue should be labeled with 'affects-x.y' or 'fixes-x.y.z', and then remove 'needs-more-info' label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/test severity/major sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.
Projects
None yet
Development

No branches or pull requests

6 participants