Resolve "(CTTC) Update NBI WebSocket endpoints to Flask-SocketIO and use gunicorn"

Closes #251 (closed)

manifests/nbiservice.yaml

@@ -23,9 +23,6 @@ spec:
   replicas: 1
   template:
     metadata:
-      annotations:
-        config.linkerd.io/skip-inbound-ports: "8762"
-        config.linkerd.io/skip-outbound-ports: "8762"
       labels:
         app: nbiservice
     spec:
@@ -36,22 +33,31 @@ spec:
           imagePullPolicy: Always
           ports:
             - containerPort: 8080
-            - containerPort: 9090
-            - containerPort: 9192
-            - containerPort: 8761
+            # Metrics disabled for now. No NBI endpoint uses it and
+            # causes "address already in use" when deploying multiple
+            # gunicorn workers.
+            #- containerPort: 9192
           env:
             - name: LOG_LEVEL
               value: "INFO"
+            - name: FLASK_ENV
+              value: "production"  # change to "development" if developing
             - name: IETF_NETWORK_RENDERER
               value: "LIBYANG"
-            - name: WS_IP_PORT
-              value: "8761"
           readinessProbe:
-            exec:
-              command: ["/bin/grpc_health_probe", "-addr=:9090"]
+            httpGet:
+              path: /healthz
+              port: 8080
+            initialDelaySeconds: 5
+            periodSeconds: 10
+            failureThreshold: 3
           livenessProbe:
-            exec:
-              command: ["/bin/grpc_health_probe", "-addr=:9090"]
+            httpGet:
+              path: /healthz
+              port: 8080
+            initialDelaySeconds: 5
+            periodSeconds: 10
+            failureThreshold: 3
           resources:
             requests:
               cpu: 50m
@@ -75,15 +81,10 @@ spec:
       protocol: TCP
       port: 8080
       targetPort: 8080
-    - name: grpc
-      protocol: TCP
-      port: 9090
-      targetPort: 9090
-    - name: metrics
-      protocol: TCP
-      port: 9192
-      targetPort: 9192
-    - name: websocket
-      protocol: TCP
-      port: 8761
-      targetPort: 8761
+    # Metrics disabled for now. No NBI endpoint uses it and
+    # causes "address already in use" when deploying multiple
+    # gunicorn workers.
+    #- name: metrics
+    #  protocol: TCP
+    #  port: 9192
+    #  targetPort: 9192