From faec1d7f463bc5ed573454a5d5cf6617fbc0f1d1 Mon Sep 17 00:00:00 2001 From: Jingtao Ren Date: Thu, 16 Nov 2017 11:18:18 -0800 Subject: [PATCH] for error case, return nil for SG --- pkg/cloudprovider/providers/azure/azure_loadbalancer.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/cloudprovider/providers/azure/azure_loadbalancer.go b/pkg/cloudprovider/providers/azure/azure_loadbalancer.go index 72ad6cfca63..80b40050ebc 100644 --- a/pkg/cloudprovider/providers/azure/azure_loadbalancer.go +++ b/pkg/cloudprovider/providers/azure/azure_loadbalancer.go @@ -798,7 +798,7 @@ func (az *Cloud) reconcileSecurityGroup(clusterName string, service *v1.Service, // Get lbIP since we make up NSG rules based on ingress IP lbIP := &lbStatus.Ingress[0].IP if lbIP == nil { - return &sg, fmt.Errorf("No load balancer IP for setting up security rules for service %s", service.Name) + return nil, fmt.Errorf("No load balancer IP for setting up security rules for service %s", service.Name) } destinationIPAddress = *lbIP }