DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #433 from ibawt/fix-scoping

Browse source 
close over the ingress variable or the last assignment will be used
This commit is contained in:
Manuel Alejandro de Brito Fontes 2017-03-13 22:16:33 -03:00 committed by GitHub
commit 7c17dc31e7
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
core/pkg/ingress/status/status.go
View file

@ -252,7 +252,7 @@ func (s *statusSync) updateStatus(newIPs []api.LoadBalancerIngress) {
continue continue
} }
go func(wg *sync.WaitGroup) { go func(wg *sync.WaitGroup, ing *extensions.Ingress) {
defer wg.Done() defer wg.Done()
ingClient := s.Client.Extensions().Ingresses(ing.Namespace) ingClient := s.Client.Extensions().Ingresses(ing.Namespace)
currIng, err := ingClient.Get(ing.Name) currIng, err := ingClient.Get(ing.Name)
@ -274,7 +274,7 @@ func (s *statusSync) updateStatus(newIPs []api.LoadBalancerIngress) {
if err != nil { if err != nil {
glog.Warningf("error updating ingress rule: %v", err) glog.Warningf("error updating ingress rule: %v", err)
} }
}(&wg) }(&wg, ing)
} }
wg.Wait() wg.Wait()