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

TestSecurityEnhancedModeRestrictedTables failed #24712

Closed
youjiali1995 opened this issue May 18, 2021 · 2 comments · Fixed by #24732
Closed

TestSecurityEnhancedModeRestrictedTables failed #24712

youjiali1995 opened this issue May 18, 2021 · 2 comments · Fixed by #24732
Assignees
Labels
component/test severity/major sig/sql-infra SIG: SQL Infra type/bug The issue is confirmed as a bug.

Comments

@youjiali1995
Copy link
Contributor

Bug Report

https://ci.pingcap.net/blue/organizations/jenkins/tidb_ghpr_unit_test/detail/tidb_ghpr_unit_test/2965/pipeline

[2021-05-18T06:18:01.299Z] ==================
[2021-05-18T06:18:01.299Z] WARNING: DATA RACE
[2021-05-18T06:18:01.299Z] Write at 0x0000071f3e78 by goroutine 176:
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/sessionctx/variable.SetSysVar()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/sessionctx/variable/sysvar.go:509 +0x129
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/util/sem.Enable()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/util/sem/sem.go:74 +0xa2
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/privilege/privileges_test.(*testPrivilegeSuite).TestSecurityEnhancedModeRestrictedTables()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/privilege/privileges/privileges_test.go:1333 +0x5e6
[2021-05-18T06:18:01.299Z]   runtime.call32()
[2021-05-18T06:18:01.299Z]       /usr/local/go/src/runtime/asm_amd64.s:539 +0x3a
[2021-05-18T06:18:01.299Z]   reflect.Value.Call()
[2021-05-18T06:18:01.299Z]       /usr/local/go/src/reflect/value.go:321 +0xd3
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).forkTest.func1()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:850 +0x9aa
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).forkCall.func1()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:739 +0x113
[2021-05-18T06:18:01.299Z] 
[2021-05-18T06:18:01.299Z] Previous read at 0x0000071f3e78 by goroutine 146:
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/domain.(*SysVarCache).RebuildSysVarCache()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/domain/sysvar_cache.go:122 +0x1bb
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/domain.(*Domain).LoadSysVarCacheLoop.func1()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/domain/domain.go:942 +0x29b
[2021-05-18T06:18:01.299Z] 
[2021-05-18T06:18:01.299Z] Goroutine 176 (running) created at:
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).forkCall()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:734 +0x4a3
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).forkTest()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:832 +0x1b9
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).doRun()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:666 +0x13a
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.(*suiteRunner).run()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:696 +0x13f
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.Run()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/run.go:150 +0x5a
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.RunAll()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/run.go:113 +0xf39
[2021-05-18T06:18:01.299Z]   github.com/pingcap/check.TestingT()
[2021-05-18T06:18:01.299Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/run.go:99 +0x765
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/privilege/privileges_test.TestT()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/privilege/privileges/privileges_test.go:49 +0x4f
[2021-05-18T06:18:01.299Z]   testing.tRunner()
[2021-05-18T06:18:01.299Z]       /usr/local/go/src/testing/testing.go:909 +0x199
[2021-05-18T06:18:01.299Z] 
[2021-05-18T06:18:01.299Z] Goroutine 146 (running) created at:
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/domain.(*Domain).LoadSysVarCacheLoop()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/domain/domain.go:916 +0x160
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/session.BootstrapSession()
[2021-05-18T06:18:01.299Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/session/session.go:2347 +0x437
[2021-05-18T06:18:01.299Z]   github.com/pingcap/tidb/privilege/privileges_test.(*testCacheSuite).SetUpSuite()
[2021-05-18T06:18:01.300Z]       /home/jenkins/agent/workspace/tidb_ghpr_unit_test/go/src/github.com/pingcap/tidb/privilege/privileges/cache_test.go:42 +0x12c
[2021-05-18T06:18:01.300Z]   runtime.call32()
[2021-05-18T06:18:01.300Z]       /usr/local/go/src/runtime/asm_amd64.s:539 +0x3a
[2021-05-18T06:18:01.300Z]   reflect.Value.Call()
[2021-05-18T06:18:01.300Z]       /usr/local/go/src/reflect/value.go:321 +0xd3
[2021-05-18T06:18:01.300Z]   github.com/pingcap/check.(*suiteRunner).runFixture.func1()
[2021-05-18T06:18:01.300Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:799 +0x195
[2021-05-18T06:18:01.300Z]   github.com/pingcap/check.(*suiteRunner).forkCall.func1()
[2021-05-18T06:18:01.300Z]       /go/pkg/mod/github.com/pingcap/check@v0.0.0-20200212061837-5e12011dc712/check.go:739 +0x113
[2021-05-18T06:18:01.300Z] ==================
@youjiali1995 youjiali1995 added type/bug The issue is confirmed as a bug. component/test labels May 18, 2021
@morgo
Copy link
Contributor

morgo commented May 18, 2021

Thanks, I will take a look tomorrow. This would have been a new issue introduced in the last day, since LoadSysVarCacheLoop was just merged.

@ti-srebot
Copy link
Contributor

Please edit this comment or add a new comment to complete the following information

Not a bug

  1. Remove the 'type/bug' label
  2. Add notes to indicate why it is not a bug

Duplicate bug

  1. Add the 'type/duplicate' label
  2. Add the link to the original bug

Bug

Note: Make Sure that 'component', and 'severity' labels are added
Example for how to fill out the template: #20100

1. Root Cause Analysis (RCA) (optional)

2. Symptom (optional)

3. All Trigger Conditions (optional)

4. Workaround (optional)

5. Affected versions

6. Fixed versions

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

Successfully merging a pull request may close this issue.

4 participants