Skip to content

Commit 79d11e7

Browse files
committed
Fix configmap update
The update of the configmap with NGINX customization was not handled properly by the Ingress Controller: - If the user had more than one Ingress resource with TLS termination conifgured, the Ingress Controller would crash. - If the user had more than one Ingress resource without TLS termination, the Ingress Controller would update the config for only one Ingress resource. The bug appeared in a552bd0. This commit fixes the bug.
1 parent 5c7be23 commit 79d11e7

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

internal/controller/controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -427,8 +427,8 @@ func (lbc *LoadBalancerController) GetManagedIngresses() ([]extensions.Ingress,
427427

428428
func (lbc *LoadBalancerController) ingressesToIngressExes(ings []extensions.Ingress) []*nginx.IngressEx {
429429
var ingExes []*nginx.IngressEx
430-
for _, ing := range ings {
431-
ingEx, err := lbc.createIngress(&ing)
430+
for i := range ings {
431+
ingEx, err := lbc.createIngress(&ings[i])
432432
if err != nil {
433433
continue
434434
}

0 commit comments

Comments
 (0)