From ced770a827b41f2eb9fe7bb04b018aff5bcf76d2 Mon Sep 17 00:00:00 2001 From: Sergey Dudoladov Date: Mon, 26 Mar 2018 11:07:32 +0200 Subject: [PATCH] Respond to code review --- pkg/cluster/k8sres.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/cluster/k8sres.go b/pkg/cluster/k8sres.go index d0b81a040..18a2253ae 100644 --- a/pkg/cluster/k8sres.go +++ b/pkg/cluster/k8sres.go @@ -697,7 +697,7 @@ func (c *Cluster) shouldCreateLoadBalancerForService(role PostgresRole, spec *sp } // `enable_load_balancer`` governs LB for a master service - // there is no equivalent operator configmap option for the replica LB + // there is no equivalent deprecated operator option for the replica LB if c.OpConfig.EnableLoadBalancer != nil { c.logger.Debugf("The operator configmap sets the deprecated `enable_load_balancer` param. Consider using the `enable_master_load_balancer` or `enable_replica_load_balancer` instead.", c.Name) return *c.OpConfig.EnableLoadBalancer @@ -736,7 +736,6 @@ func (c *Cluster) generateService(role PostgresRole, spec *spec.PostgresSpec) *v // safe default value: lock load balancer to only local address unless overridden explicitly. sourceRanges := []string{localHost} - // source ranges are the same for balancers for master and replica services allowedSourceRanges := spec.AllowedSourceRanges if len(allowedSourceRanges) >= 0 { sourceRanges = allowedSourceRanges