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

fix srs-server helm chart port config problem. #6

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 15 additions & 7 deletions srs-server/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,19 +36,19 @@ spec:
command: ["./objs/srs", "-e"]
ports:
- name: rtmp
containerPort: {{ .Values.service.rtmp }}
containerPort: 1935
protocol: TCP
- name: http
containerPort: {{ .Values.service.http }}
containerPort: 8080
protocol: TCP
- name: api
containerPort: {{ .Values.service.api }}
containerPort: 1985
protocol: TCP
- name: srt
containerPort: {{ .Values.service.srt }}
containerPort: 10080
protocol: UDP
- name: rtc
containerPort: {{ .Values.service.rtc }}
containerPort: 8000
protocol: UDP
env:
# The general default config.
Expand Down Expand Up @@ -86,12 +86,20 @@ spec:
value: "off"
livenessProbe:
httpGet:
path: /
port: http
path: /api/v1/versions
port: api
periodSeconds: 10
readinessProbe:
httpGet:
path: /
port: http
periodSeconds: 10
startupProbe:
httpGet:
path: /api/v1/versions
port: api
failureThreshold: 3
periodSeconds: 10
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{- with .Values.nodeSelector }}
Expand Down
2 changes: 2 additions & 0 deletions srs-server/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ image:
tag: ""
pullPolicy: IfNotPresent

# simple set replicaCount more than one don't make sense, because the default srs config is stateful app.
replicaCount: 1

# The configuration for SRS can be overridden by environment variables.
Expand Down Expand Up @@ -107,6 +108,7 @@ resources: {}
# cpu: 100m
# memory: 128Mi

# simple open autoscaling don't make sense, because the default srs config is stateful app.
autoscaling:
enabled: false
minReplicas: 1
Expand Down