From 6d28bee388c459999bc71de29677ef190c451d6c Mon Sep 17 00:00:00 2001 From: Dobes Vandermeer Date: Wed, 22 May 2019 13:40:16 -0700 Subject: [PATCH] Update matchAllDomainsNumLabels and numLabelsSpecificMatch These variables seem like they should be updated when updating the thing they are supposed to be derived from. Signed-off-by: Dobes Vandermeer --- pkg/controller/acmeorders/sync.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pkg/controller/acmeorders/sync.go b/pkg/controller/acmeorders/sync.go index 086834566..4ee67d786 100644 --- a/pkg/controller/acmeorders/sync.go +++ b/pkg/controller/acmeorders/sync.go @@ -495,6 +495,7 @@ func determineSolverConfigToUse(candidates []cmapi.ACMEChallengeSolver, authz *a if len(d.Selector.MatchLabels) > matchAllDomainsNumLabels || matchAll == nil { matchAll = &d matchAllToSolve = acmech + matchAllDomainsNumLabels = len(d.Selector.MatchLabels) } } for _, dom := range d.Selector.DNSNames { @@ -504,6 +505,7 @@ func determineSolverConfigToUse(candidates []cmapi.ACMEChallengeSolver, authz *a if len(d.Selector.MatchLabels) > numLabelsSpecificMatch || specificMatch == nil { specificMatch = &d specificMatchToSolve = acmech + numLabelsSpecificMatch = len(d.Selector.MatchLabels) break } }