Browse Source

fix: Correct admin server port in Helm worker deployment (#7872)

The worker deployment was incorrectly passing the admin gRPC port (33646)
to the -admin flag. However, the SeaweedFS worker command automatically
calculates the gRPC port by adding 10000 to the HTTP port provided.

This caused workers to attempt connection to port 43646 (33646 + 10000)
instead of the correct gRPC port 33646 (23646 + 10000).

Changes:
- Update worker-deployment.yaml to use admin.port instead of admin.grpcPort
- Workers now correctly connect to admin HTTP port, allowing the binary
  to calculate the gRPC port automatically

Fixes workers failing with:
"dial tcp <admin-ip>:43646: connect: no route to host"

Related:
- Worker code: weed/pb/grpc_client_server.go:272 (grpcPort = port + 10000)
- Worker docs: weed/command/worker.go:36 (admin HTTP port + 10000)
pull/7874/head
Sheya Bernstein 2 days ago
committed by GitHub
parent
commit
7f611f5d3a
No known key found for this signature in database GPG Key ID: B5690EEEBB952194
  1. 2
      k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml

2
k8s/charts/seaweedfs/templates/worker/worker-deployment.yaml

@ -134,7 +134,7 @@ spec:
{{- if .Values.worker.adminServer }} {{- if .Values.worker.adminServer }}
-admin={{ .Values.worker.adminServer }} \ -admin={{ .Values.worker.adminServer }} \
{{- else }} {{- else }}
-admin={{ template "seaweedfs.name" . }}-admin.{{ .Release.Namespace }}:{{ .Values.admin.grpcPort }} \
-admin={{ template "seaweedfs.name" . }}-admin.{{ .Release.Namespace }}:{{ .Values.admin.port }} \
{{- end }} {{- end }}
-capabilities={{ .Values.worker.capabilities }} \ -capabilities={{ .Values.worker.capabilities }} \
-maxConcurrent={{ .Values.worker.maxConcurrent }} \ -maxConcurrent={{ .Values.worker.maxConcurrent }} \

Loading…
Cancel
Save