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

Does not work for NodePort #75

Closed
CASL0 opened this issue Jan 18, 2024 · 3 comments
Closed

Does not work for NodePort #75

CASL0 opened this issue Jan 18, 2024 · 3 comments

Comments

@CASL0
Copy link
Contributor

CASL0 commented Jan 18, 2024

Describe your Issue

It doesn't work for NodePort, because NodePort cannot use the same port.

Logs and Errors

Error: UPGRADE FAILED: cannot patch "myrelease-turn-udp" with kind Service: Service "myrelease-turn-udp" is invalid: spec.ports[0].nodePort: Invalid value: 30030: provided port is already allocated

Describe your Environment

  • Kubernetes distribution: kind

  • Helm Version (or App that manages helm): 3.13.3

  • Helm Chart Version: 4.3.0

  • values.yaml:

coturn:
  realm: "localhost"
  auth:
    password: "password"
  ports:
    tlsListening: 30030
    listening: 30031
service:
  type: NodePort

Additional context, if any

a typo in ".Values.coturn.ports" as ".Values.coturn.port," so you need to correct it together.

@jessebot
Copy link
Collaborator

sorry for missing this. checking it out now!

@jessebot
Copy link
Collaborator

closed by #76 and will be merged as part of the release associated with #85

@jessebot
Copy link
Collaborator

This fix is now live with 5.0.0. Thanks, @CASL0 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants