From 9d102e3bff7a02e03b279329edc5e607841c20e7 Mon Sep 17 00:00:00 2001 From: Rijnard van Tonder Date: Wed, 10 Oct 2018 16:05:48 -0400 Subject: [PATCH] The nil check before the range loop is redundant Signed-off-by: Rijnard van Tonder --- discovery/gce/gce.go | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/discovery/gce/gce.go b/discovery/gce/gce.go index 4a9cae891..d0d75de06 100644 --- a/discovery/gce/gce.go +++ b/discovery/gce/gce.go @@ -244,11 +244,9 @@ func (d *Discovery) refresh() (tg *targetgroup.Group, err error) { } // GCE labels are key-value pairs that group associated resources - if inst.Labels != nil { - for key, value := range inst.Labels { - name := strutil.SanitizeLabelName(key) - labels[gceLabelLabel+model.LabelName(name)] = model.LabelValue(value) - } + for key, value := range inst.Labels { + name := strutil.SanitizeLabelName(key) + labels[gceLabelLabel+model.LabelName(name)] = model.LabelValue(value) } if len(priIface.AccessConfigs) > 0 {