Description of the change
TURNS cannot establish a connection over UDP, it requires TCP. The current chart creates two separate services:
- One for UDP
- One for TCP
This is unnecessary. Instead, a single Service can handle both protocols on the same ports. Here's what works:
apiVersion:v1kind:Servicemetadata:name:coturn-lbnamespace:coturnspec:type:LoadBalancerexternalTrafficPolicy:Localports:- name:turn-udpport:3478targetPort:3478protocol:UDP- name:turn-tcpport:3478targetPort:3478protocol:TCP- name:turn-tls-udpport:5349targetPort:5349protocol:UDP- name:turn-tls-tcpport:5349targetPort:5349protocol:TCPselector:app.kubernetes.io/name:coturn-coturnapp.kubernetes.io/instance:coturnThe chart should be updated to use a single service instead of two.
Benefits
avoids redundancy, and ensures TURNS works properly.
Possible drawbacks
Unsure.
Additional information
This was originally suggested in https://github.com/small-hack/coturn-chart/issues/110#issuecomment-2652207121 and ported here to codeberg.org since we're archiving the github.com repo.
## Description of the change
TURNS cannot establish a connection over UDP, it requires TCP. The current chart creates two separate services:
- One for UDP
- One for TCP
This is unnecessary. Instead, a single Service can handle both protocols on the same ports. Here's what works:
```yaml
apiVersion: v1
kind: Service
metadata:
name: coturn-lb
namespace: coturn
spec:
type: LoadBalancer
externalTrafficPolicy: Local
ports:
- name: turn-udp
port: 3478
targetPort: 3478
protocol: UDP
- name: turn-tcp
port: 3478
targetPort: 3478
protocol: TCP
- name: turn-tls-udp
port: 5349
targetPort: 5349
protocol: UDP
- name: turn-tls-tcp
port: 5349
targetPort: 5349
protocol: TCP
selector:
app.kubernetes.io/name: coturn-coturn
app.kubernetes.io/instance: coturn
```
The chart should be updated to use a single service instead of two.
## Benefits
avoids redundancy, and ensures TURNS works properly.
## Possible drawbacks
Unsure.
## Additional information
This was originally suggested in https://github.com/small-hack/coturn-chart/issues/110#issuecomment-2652207121 and ported here to codeberg.org since we're archiving the github.com repo.