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

panic in TestRollingPolicy_Add #7386

Closed
lhy1024 opened this issue Nov 16, 2023 · 2 comments · Fixed by #7511
Closed

panic in TestRollingPolicy_Add #7386

lhy1024 opened this issue Nov 16, 2023 · 2 comments · Fixed by #7511
Assignees
Labels
type/ci The issue is related to CI.

Comments

@lhy1024
Copy link
Contributor

lhy1024 commented Nov 16, 2023

Flaky Test

Which jobs are failing

2023-11-15T17:04:55.9143408Z --- FAIL: TestRollingPolicy_Add (0.30s)
2023-11-15T17:04:55.9145312Z     --- FAIL: TestRollingPolicy_Add/test_policy_add#01 (0.10s)
2023-11-15T17:04:55.9147043Z panic: runtime error: index out of range [0] with length 0 [recovered]
2023-11-15T17:04:55.9148402Z 	panic: runtime error: index out of range [0] with length 0
2023-11-15T17:04:55.9149534Z 
2023-11-15T17:04:55.9150160Z goroutine 34 [running]:
2023-11-15T17:04:55.9151067Z testing.tRunner.func1.2({0x720020, 0xc00011c000})
2023-11-15T17:04:55.9152656Z 	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1545 +0x366
2023-11-15T17:04:55.9154041Z testing.tRunner.func1()
2023-11-15T17:04:55.9155164Z 	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1548 +0x630
2023-11-15T17:04:55.9156303Z panic({0x720020?, 0xc00011c000?})
2023-11-15T17:04:55.9157696Z 	/opt/hostedtoolcache/go/1.21.3/x64/src/runtime/panic.go:920 +0x270
2023-11-15T17:04:55.9159081Z github.com/tikv/pd/pkg/window.TestRollingPolicy_Add.func1(0x0?)
2023-11-15T17:04:55.9160653Z 	/home/runner/work/pd/pd/pkg/window/policy_test.go:70 +0x4af
2023-11-15T17:04:55.9161925Z testing.tRunner(0xc000222000, 0xc000216000)
2023-11-15T17:04:55.9163123Z 	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1595 +0x239
2023-11-15T17:04:55.9164267Z created by testing.(*T).Run in goroutine 3
2023-11-15T17:04:55.9165589Z 	/opt/hostedtoolcache/go/1.21.3/x64/src/testing/testing.go:1648 +0x82b
2023-11-15T17:04:55.9166765Z FAIL	github.com/tikv/pd/pkg/window	8.319s

CI link

https://github.com/lhy1024/pd/actions/runs/6880527737/job/18714932638

Reason for failure (if possible)

Anything else

@lhy1024 lhy1024 added the type/ci The issue is related to CI. label Nov 16, 2023
@CabinfeverB
Copy link
Member

I run this test 500 times in local env. I think that time.Sleep in CI environment has extra time consumption.

@CabinfeverB CabinfeverB self-assigned this Nov 16, 2023
@rleungx
Copy link
Member

rleungx commented Nov 23, 2023

@ti-chi-bot ti-chi-bot bot closed this as completed in #7511 Dec 8, 2023
ti-chi-bot bot added a commit that referenced this issue Dec 8, 2023
close #7386

Signed-off-by: Cabinfever_B <cabinfeveroier@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/ci The issue is related to CI.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants