Fix go routine leaks

Close the channel instead of write to it to avoid hanging goroutine.
This commit is contained in:
Msk 2022-11-30 23:50:59 +08:00
parent 57eb5d631c
commit 1444f089fc

View File

@ -66,10 +66,10 @@ func (r *EtcdMigrateServer) Start(version *EtcdVersion) error {
} }
interval := time.NewTicker(time.Millisecond * 500) interval := time.NewTicker(time.Millisecond * 500)
defer interval.Stop() defer interval.Stop()
done := make(chan bool, 1) done := make(chan bool)
go func() { go func() {
time.Sleep(time.Minute * 2) time.Sleep(time.Minute * 2)
done <- true close(done)
}() }()
for { for {
select { select {
@ -108,7 +108,7 @@ func (r *EtcdMigrateServer) Stop() error {
timedout := make(chan bool) timedout := make(chan bool)
go func() { go func() {
time.Sleep(gracefulWait) time.Sleep(gracefulWait)
timedout <- true close(timedout)
}() }()
go func() { go func() {
select { select {
@ -121,7 +121,7 @@ func (r *EtcdMigrateServer) Stop() error {
} }
}() }()
err = r.cmd.Wait() err = r.cmd.Wait()
stopped <- true close(stopped)
if exiterr, ok := err.(*exec.ExitError); ok { if exiterr, ok := err.(*exec.ExitError); ok {
klog.Infof("etcd server stopped (signal: %s)", exiterr.Error()) klog.Infof("etcd server stopped (signal: %s)", exiterr.Error())
// stopped // stopped