[DCAE][HVVES] Make HVVES collector SM compatible 55/133255/5
authorAndreas Geissler <andreas-geissler@telekom.de>
Tue, 14 Feb 2023 16:27:42 +0000 (17:27 +0100)
committerAndreas Geissler <andreas-geissler@telekom.de>
Fri, 17 Feb 2023 08:47:09 +0000 (09:47 +0100)
As we have problems to send binary data to the HVVES collector
the support for service appProtocol attribute is required to
fix the hvves service protocol to "tcp" and to disable the
port for sidecar usage

Issue-ID: OOM-3076

Signed-off-by: Andreas Geissler <andreas-geissler@telekom.de>
Change-Id: Ied6ca4d4d8cd088c8ec0e0ee13068dfdbfb38461

kubernetes/common/common/templates/_service.tpl
kubernetes/dcaegen2-services/components/dcae-hv-ves-collector/values.yaml

index 20d2ebe..4b6e0a1 100644 (file)
@@ -126,6 +126,9 @@ labels: {{- include "common.labels" (dict "labels" $labels "dot" $dot) | nindent
 {{-       else }}
   protocol: TCP
 {{-       end }}
+{{-       if $port.app_protocol }}
+  appProtocol: {{ $port.app_protocol }}
+{{-       end }}
 {{-       if $port.port_protocol }}
   name: {{ printf "%ss-%s" $port.port_protocol $port.name }}
 {{-       else }}
@@ -139,6 +142,9 @@ labels: {{- include "common.labels" (dict "labels" $labels "dot" $dot) | nindent
 {{-       else }}
   protocol: {{ default "TCP" $port.l4_protocol  }}
 {{-       end }}
+{{-       if $port.app_protocol }}
+  appProtocol: {{ $port.app_protocol }}
+{{-       end }}
 {{-       if $port.port_protocol }}
   name: {{ printf "%s-%s" $port.port_protocol $port.name }}
 {{-       else }}
@@ -157,6 +163,9 @@ labels: {{- include "common.labels" (dict "labels" $labels "dot" $dot) | nindent
 {{-         else }}
   protocol: {{ default "TCP" $port.l4_protocol  }}
 {{-         end }}
+{{-       if $port.app_protocol }}
+  appProtocol: {{ $port.app_protocol }}
+{{-       end }}
 {{-         if $port.port_protocol }}
   name: {{ printf "%s-%s" $port.port_protocol $port.name }}
 {{-         else }}
index f788ab0..351998a 100644 (file)
@@ -103,14 +103,21 @@ readiness:
   command:
   - /opt/ves-hv-collector/healthcheck.sh
 
+# since there are problems receiving binary data via the sidecar
+# the service port is excluded in the sidecar processing
+podAnnotations:
+  traffic.sidecar.istio.io/excludeInboundPorts: "6061"
+  traffic.sidecar.istio.io/includeInboundPorts: '*'
+
 # service configuration
 service:
   type: NodePort
   name: dcae-hv-ves-collector
   ports:
-    - name: http
+    - name: tcp
       port: 6061
-      port_protocol: http
+      port_protocol: tcp
+      app_protocol: tcp
       nodePort: 22
 
 ingress: