Skip to content

Commit 1504d25

Browse files
committed
🐛elb: fix printing info message with ELB name
Fixes the following message being printed: ``` loadbalancer.go:1799: Pre-existing NLB %s without security groups, cannot reconcile security groups ``` to actually include the load balancer's name.
1 parent 888c659 commit 1504d25

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/cloud/services/elb/loadbalancer.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1788,7 +1788,7 @@ func shouldReconcileSGs(scope scope.ELBScope, lb *infrav1.LoadBalancer, specSGs
17881788
// Once created without a security group, the NLB can never have any added.
17891789
// (https://docs.aws.amazon.com/elasticloadbalancing/latest/network/load-balancer-security-groups.html)
17901790
if lb.LoadBalancerType == infrav1.LoadBalancerTypeNLB && len(lb.SecurityGroupIDs) == 0 {
1791-
scope.Info("Pre-existing NLB %s without security groups, cannot reconcile security groups.", lb.Name)
1791+
scope.Info("Pre-existing NLB without security groups, cannot reconcile security groups.", "elb-name", lb.Name)
17921792
return false
17931793
}
17941794
if !sets.NewString(lb.SecurityGroupIDs...).Equal(sets.NewString(specSGs...)) {

0 commit comments

Comments
 (0)