Test to ensure otlp ports in collector e2e (#945)

* Add a simple test to ensure we are opening the correct ports for otlp receivers Signed-off-by: Kevin Earls <[email protected]> * Fixed typos, yaml formatting Signed-off-by: Kevin Earls <[email protected]> * Add otlp port checks to existing smoke test Signed-off-by: Kevin Earls <[email protected]>
open-telemetry · Jun 27, 2022 · ce77f82 · ce77f82
1 parent 359478c
commit ce77f82
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 1 deletion.
diff --git a/tests/e2e/smoke-simplest/00-assert.yaml b/tests/e2e/smoke-simplest/00-assert.yaml
@@ -18,6 +18,21 @@ spec:
     port: 14250
     protocol: TCP
     targetPort: 14250
+  - appProtocol: grpc
+    name: otlp-grpc
+    port: 4317
+    protocol: TCP
+    targetPort: 4317
+  - appProtocol: http
+    name: otlp-http
+    port: 4318
+    protocol: TCP
+    targetPort: 4318
+  - appProtocol: http
+    name: otlp-http-legacy
+    port: 55681
+    protocol: TCP
+    targetPort: 4318
 
 ---
 
@@ -32,3 +47,18 @@ spec:
     port: 14250
     protocol: TCP
     targetPort: 14250
+  - appProtocol: grpc
+    name: otlp-grpc
+    port: 4317
+    protocol: TCP
+    targetPort: 4317
+  - appProtocol: http
+    name: otlp-http
+    port: 4318
+    protocol: TCP
+    targetPort: 4318
+  - appProtocol: http
+    name: otlp-http-legacy
+    port: 55681
+    protocol: TCP
+    targetPort: 4318
diff --git a/tests/e2e/smoke-simplest/00-install.yaml b/tests/e2e/smoke-simplest/00-install.yaml
@@ -8,6 +8,10 @@ spec:
       jaeger:
         protocols:
           grpc:
+      otlp:
+        protocols:
+          grpc:
+          http:
     processors:
 
     exporters:
@@ -16,6 +20,6 @@ spec:
     service:
       pipelines:
         traces:
-          receivers: [jaeger]
+          receivers: [jaeger,otlp]
           processors: []
           exporters: [logging]