seaweedfs: Lance ingress and worker URL still point at the old S3 Service name
Los mantenedores suelen responder en 3 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- helm, kubernetes, yaml
- Área
- infrastructure
Línea de trabajo
Lee s3-service-name-consumers.patch y los casos existentes en tests/s3_service_name_consumers_test.yaml. Actualiza las referencias al backend de ingress de Lance y a la URL del worker que se muestran en el issue para que usen el Service de S3 renombrado; después, añade un caso con un lancePort distinto de cero y un fullnameOverride sobrescrito. El trabajo estará terminado cuando los nombres renderizados usen el Service renombrado y pase la prueba.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The Lance catalog consumers were never moved to the renamed S3 Service, so they point at a name that may not exist.
The S3 Service is renamed to <name>-s3 by s3-service-name.patch, and s3-service-name-consumers.patch moves its consumers to that name. The S3 ingress, the Iceberg ingress and the COSI endpoint were moved. The Lance catalog came with chart 4.45, and its two consumers still build the name with seaweedfs.componentName, which gives <fullname>-s3:
- the Lance ingress backend, s3-lance-ingress.yaml#L8-L12
- the Lance URL the worker gets from
seaweedfs.worker.lanceNamespaceUrl, _helpers.tpl#L156
Nothing breaks today. values.yaml sets s3.lancePort: 0, and both consumers render only with a non-zero port. On top of that, fullnameOverride: seaweedfs makes both names the same string. Turn the catalog on under a different fullname and the Lance ingress points at a Service that does not exist, so it returns 503, and the worker gets a URL that does not resolve.
Fix: add both places to s3-service-name-consumers.patch with the same printf "%s-s3" (include "seaweedfs.name" .) form the other consumers use. A case in tests/s3_service_name_consumers_test.yaml should set a non-zero lancePort and override fullnameOverride, like the existing cases do.
- Lenguaje dominante
- Go
- Estrellas
- 2.2k
- Forks
- 209
- Merge medio
- 3 d 9 h
- PR fusionados (30 d)
- 262
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cozystack/cozystack
-
area/release kind/bug kind/documentation triage/needs-triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
cozystack/cozystack#4892 · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
triage/needs-triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 64/100
cozystack/cozystack#4886 · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
area/testing kind/cleanup priority/backlog triage/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
cozystack/cozystack#4882 · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
testing: helm-unittest's isNotNullOrEmpty passes when the path does not existPosiblemente ocupada @Shxshxnkk1 la tomó hace 4 días. Abiertoarea/testing kind/bug triage/needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
cozystack/cozystack#4841 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 3 días
-
chore(cilium): drop externalIPs.enabled and nodePort.enabled, the Cilium chart does not read themAbiertoarea/cilium kind/cleanup triage/needs-triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
cozystack/cozystack#4816 · 1 reacción ·
Los mantenedores suelen responder en 3 días
Todos los issues de cozystack/cozystack
Issues similares
-
TOTP enrollment returns 500 for phone-only users (empty TOTP account name)Posiblemente ocupada @emiliosandoval4 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 60/100
Los mantenedores suelen responder en 1 día
-
8.9 to 8.10 Helm upgrade guide still says deprecated keys are removed in chart v16 (Camunda 8.11)Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
camunda/camunda-platform-helm#7559 ·
Los mantenedores suelen responder en 1 día
-
docs: namespaceRegexes example teaches prefix matching, but regexes are matched as a substringPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertotriage/pending-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
external-secrets/external-secrets#7117 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ENTERPILOT/GoModel#1183 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
createEtcdKV fails on a single transient etcd endpoint sync timeoutPosiblemente ocupada @vldmit la tomó hoy. Abiertocontribution
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día