metallb: Remove additional nodeSelector in deployment

- Fix the regression added in the previous update by removing the
  extraneous nodeSelector field.

- Move the nodeSelector golang template block down so that in the next
  update cycle we don't encounter this issue.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>

pkg/components/metallb/manifests.go

@@ -248,12 +248,6 @@ spec:
         app: metallb
         component: controller
     spec:
-      {{- if .ControllerNodeSelectors }}
-      nodeSelector:
-        {{- range $key, $value := .ControllerNodeSelectors }}
-        {{ $key }}: "{{ $value }}"
-        {{- end }}
-      {{- end }}
       containers:
       - args:
         - --port=7472
@@ -274,8 +268,12 @@ spec:
             drop:
             - all
           readOnlyRootFilesystem: true
+      {{- if .ControllerNodeSelectors }}
       nodeSelector:
-        beta.kubernetes.io/os: linux
+        {{- range $key, $value := .ControllerNodeSelectors }}
+        {{ $key }}: "{{ $value }}"
+        {{- end }}
+      {{- end }}
       securityContext:
         runAsNonRoot: true
         runAsUser: 65534