Configure Ingress for both HTTP and GRPC otelCollector

[c5racing]

This doc is great for getting self hosted running behind NGINX: https://signoz.io/docs/tutorial/setting-up-tls-for-signoz. It works out of the box for GRPC and the web front end. Unfortunately, the HTTP collector doesn't receive ingress traffic.

Can I get some assistance in setting up the ingress traffic for the HTTP collector?

If I configure my values.yaml file as the following, my GRPC quits working:

ingress: enabled: true className: nginx hosts: - host: signoz-ingest.example.com paths: - path: /v1/traces pathType: ImplementationSpecific port: 4318 # HTTP ingestion - path: /v1/metrics pathType: ImplementationSpecific port: 4318 # HTTP ingestion - path: /v1/logs pathType: ImplementationSpecific port: 4318 # HTTP ingestion - path: / pathType: ImplementationSpecific port: 4317 # GRPC ingestion tls: - secretName: signoz-ingest-tls-cert hosts: - signoz-ingest.example.com annotations: cert-manager.io/cluster-issuer: letsencrypt-production nginx.ingress.kubernetes.io/ssl-redirect: "true"

If I add back in nginx.ingress.kubernetes.io/backend-protocol: "GRPC" under annotataions, then my HTTP quits working

[therealpandey]

The implementation idea is simple. We need to create 2 ingresses: One for grpc, one for http:

Spec for ingress/signoz-otel-collector-grpc:

metadata:
  annotations:
    nginx.ingress.kubernetes.io/backend-protocol: GRPC
spec:
  ingressClassName: nginx
  rules:
  - host: <host>
    http:
      paths:
      - backend:
          service:
            name: signoz-otel-collector
            port:
              number: 4317
        path: /opentelemetry.proto.collector.logs.v1.LogsService/Export
        pathType: Exact
      - backend:
          service:
            name: signoz-otel-collector
            port:
              number: 4317
        path: /opentelemetry.proto.collector.metrics.v1.MetricsService/Export
        pathType: Exact
      - backend:
          service:
            name: signoz-otel-collector
            port:
              number: 4317
        path: /opentelemetry.proto.collector.trace.v1.TraceService/Export
        pathType: Exact

Spec for ingress/signoz-otel-collector-http:

spec:
  ingressClassName: nginx
  rules:
  - host: <host>
    http:
      paths:
      - backend:
          service:
            name:  signoz-otel-collector
            port:
              number: 4318
        path: /v1/logs
        pathType: Exact
      - backend:
          service:
            name:  signoz-otel-collector
            port:
              number: 4318
        path: /v1/metrics
        pathType: Exact
      - backend:
          service:
            name:  signoz-otel-collector
            port:
              number: 4318
        path: /v1/traces
        pathType: Exact

[therealpandey]

@c5racing, would you be willing to work on a PR for this?

[suyash-811]

Hey, is this issue available to work on? If yes, i would like to work on this

[therealpandey]

Hey @suyash-811,
Yes it's available to work on.

[LylaB]

Hi👋

If this issue is still open, I’d like to work on it as my first contribution.
Please let me know if this is still available.

[Nageshbansal]

Hey @LylaB Please go ahead with working on this issue

[suyash-811]

Hi @Nageshbansal

I've already created a PR with the changes ( referenced above) and am waiting for a final review on the changes since the last few months. Maybe you could take a look?