Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

istio protocol selection #696

Closed
binjip978 opened this issue Feb 9, 2022 · 3 comments
Closed

istio protocol selection #696

binjip978 opened this issue Feb 9, 2022 · 3 comments
Labels
area:collector Issues for deploying collector good first issue Good for newcomers

Comments

@binjip978
Copy link
Contributor

Hi, we are using opentelemetry collector with Istio and facing issue with protocol selection on istio side. According to istio documentation protocol selection can be configured in two ways:

  • By the name of the port, in format: "name: <protocol>[-<suffix>]"
  • By specifying appProtocol, work with Kubernetes 1.18+

Unfortunately for us collector specify protocol name in a service as "otlp-grpc", and istio can't deduce and pick grpc, it would work if name was "grpc-oltp".

So, there are two solution for this problem:

  • rename oltp-grpc to grpc-oltp, but it's probably to big to change right now, but i'm not totally sure about that.
  • add appProtocol field to OLTP/Jaeger receiver parsers Ports function, right now is currently not used and specify protocol name there, and it should look like this:
    ports:
    - appProtocol: grpc
      name: otlp-grpc
      port: 4317
      protocol: TCP
      targetPort: 4317

Any feedback are welcome, and i can provide a patch if any approach is suitable for the project.

@pavolloffay
Copy link
Member

pavolloffay commented Feb 9, 2022

Defining the correct appProtocol in service seems like the best way forward to me.

@pavolloffay pavolloffay added the area:collector Issues for deploying collector label Feb 9, 2022
@jpkrohling jpkrohling added the good first issue Good for newcomers label Feb 9, 2022
@binjip978
Copy link
Contributor Author

thx, for the fast feedback, will create a PR.

@pavolloffay
Copy link
Member

Done in #704

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:collector Issues for deploying collector good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants