From 301abddc7218c11003481e03d0ac1abface37d25 Mon Sep 17 00:00:00 2001 From: Gyu-Ho Lee Date: Tue, 13 Dec 2016 04:58:23 -0800 Subject: [PATCH] etcd-tester: cancel lease stream; fix OOM panic It was never closing lease keep-alive streams, leaking memory. Fix OOM panics in etcd-tester (after 1K rounds). --- tools/functional-tester/etcd-tester/lease_stresser.go | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/functional-tester/etcd-tester/lease_stresser.go b/tools/functional-tester/etcd-tester/lease_stresser.go index 963a902a8..cd08d8cfa 100644 --- a/tools/functional-tester/etcd-tester/lease_stresser.go +++ b/tools/functional-tester/etcd-tester/lease_stresser.go @@ -266,6 +266,7 @@ func (ls *leaseStresser) keepLeaseAlive(leaseID int64) { defer ls.aliveWg.Done() ctx, cancel := context.WithCancel(ls.ctx) stream, err := ls.lc.LeaseKeepAlive(ctx) + defer func() { cancel() }() for { select { case <-time.After(500 * time.Millisecond):