[apisix-ingress-controller] branch master updated: fix: timer leak memory (#591)

[zh...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zhangjintao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git


The following commit(s) were added to refs/heads/master by this push:
     new 5635652  fix: timer leak memory (#591)
5635652 is described below

commit 5635652865ef965db58794b08e2fe37ddc9c08e3
Author: yuanfeng0905 <yu...@gmail.com>
AuthorDate: Wed Aug 4 14:58:16 2021 +0800

    fix: timer leak memory (#591)
    
    Co-authored-by: Leon <yu...@haituncun.com>
---
 pkg/ingress/controller.go | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/pkg/ingress/controller.go b/pkg/ingress/controller.go
index 3c3d566..9897256 100644
--- a/pkg/ingress/controller.go
+++ b/pkg/ingress/controller.go
@@ -601,11 +601,13 @@ func (c *Controller) syncUpstreamNodesChangeToCluster(ctx context.Context, clust
 
 func (c *Controller) checkClusterHealth(ctx context.Context, cancelFunc context.CancelFunc) {
 	defer cancelFunc()
+	t := time.NewTicker(5 * time.Second)
+	defer t.Stop()
 	for {
 		select {
 		case <-ctx.Done():
 			return
-		case <-time.After(5 * time.Second):
+		case <-t.C:
 		}
 
 		err := c.apisix.Cluster(c.cfg.APISIX.DefaultClusterName).HealthCheck(ctx)