Federated NGINX Service - type: NodePort

[jamesbuckett]

The file kubernetes-cluster-federation/services/nginx.yaml
has
type: NodePort
Should be
type: LoadBalancer

File should read :

apiVersion: v1
kind: Service
metadata:
  labels:
    app: nginx
  name: nginx
spec:
  ports:
  - port: 80
    protocol: TCP
    targetPort: 80
    name: http
  selector:
    app: nginx
  type: LoadBalancer

[metalmatze]

This should really be updated. I just tried to create the service and nothing happend on the federation side of things. It's misleading like this...

[jbelinchonm]

Issue regards to documentation. Service was changed to NodePort in PR #42 and merged. I guess it would be nice if someone updates the docs and includes the federated ingress bit too.