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

goroutine leak: goroutine increase during the long-term operation #9047

Closed
bufferflies opened this issue Feb 8, 2025 · 0 comments · Fixed by #9048
Closed

goroutine leak: goroutine increase during the long-term operation #9047

bufferflies opened this issue Feb 8, 2025 · 0 comments · Fixed by #9048
Assignees
Labels
affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.

Comments

@bufferflies
Copy link
Contributor

bufferflies commented Feb 8, 2025

Bug Report

The count of pd goroutine continue increase in long-term cluster

What did you do?

Trigger condition: remove scheduler

#/bin/bash

for i in $(seq 1 50); do
tiup ctl:v6.5.6 pd -u  http://127.0.0.1:2379 scheduler remove balance-leader-scheduler
sleep 10
tiup ctl:v6.5.6 pd -u  http://127.0.0.1:2379 scheduler add balance-leader-scheduler
sleep 20
done
Image

root cause:

decoder := schedule.ConfigSliceDecoder(schedulerCfg.Type, schedulerCfg.Args)
tmp, err := schedule.CreateScheduler(schedulerCfg.Type, schedule.NewOperatorController(c.ctx, nil, nil), storage.NewStorageWithMemoryBackend(), decoder)
if err != nil {
return err
}

What did you expect to see?

The count of goroutine is steady

What did you see instead?

What version of PD are you using (pd-server -V)?

v6.5.6

@bufferflies bufferflies added the type/bug The issue is confirmed as a bug. label Feb 8, 2025
@bufferflies bufferflies self-assigned this Feb 8, 2025
@bufferflies bufferflies added affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. severity/major labels Feb 8, 2025
@ti-chi-bot ti-chi-bot bot closed this as completed in c92da83 Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-6.1 This bug affects the 6.1.x(LTS) versions. affects-6.5 This bug affects the 6.5.x(LTS) versions. affects-7.1 This bug affects the 7.1.x(LTS) versions. severity/major type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant