Merge pull request #587 from chechiachang/feature/add-grpc-port-config-in-helm-charts

feature: add grpc port and config in helm chart

charts/bbgo/templates/deployment.yaml

@@ -51,6 +51,8 @@ spec:
           {{- end }}
           {{- if .Values.grpc.enabled }}
           - "--enable-grpc"
+          - "--grpc-bind"
+          - {{ .Values.grpc.port | default 50051 | quote }}
           {{- end }}
           {{- if .Values.debug.enabled }}
           - "--debug"
@@ -62,6 +64,11 @@ spec:
             containerPort: 8080
             protocol: TCP
           {{- end }}
+          {{- if .Values.grpc.enabled }}
+          - name: grpc
+            containerPort: {{ .Values.grpc.port | default 50051 }}
+            protocol: TCP
+          {{- end }}
           {{- if .Values.metrics.enabled }}
           - name: metrics
             containerPort: 9090

charts/bbgo/templates/service.yaml

@@ -11,5 +11,11 @@ spec:
       targetPort: http
       protocol: TCP
       name: http
+    {{- if .Values.grpc.enabled }}
+    - port: {{ .Values.grpc.port | default 50051 }}
+      targetPort: grpc
+      protocol: TCP
+      name: grpc
+    {{- end }}
   selector:
     {{- include "bbgo.selectorLabels" . | nindent 4 }}

charts/bbgo/values.yaml

@@ -76,6 +76,7 @@ metrics:
 
 grpc:
   enabled: false
+  port: 50051
 
 debug:
   enabled: false