[Bug] Helm chart: metrics.enabled=false still adds kube-rbac-proxy
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- helm, kubernetes
- Área
- devops
Línea de trabajo
The kube-rbac-proxy sidecar condition lives in templates/manager.yaml of the Helm chart and checks only metrics.disableAuth, not metrics.enabled. Pull request #690 is already open against this issue, so read it and coordinate there before starting. Done means helm template with --set metrics.enabled=false renders no kube-rbac-proxy container, which the grep in the reproduction confirms.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What are you really trying to do?
Run the controller with its metrics endpoint turned off.
Describe the bug
metrics.enabled: false still adds the kube-rbac-proxy sidecar. Its condition in templates/manager.yaml checks only metrics.disableAuth.
Minimal Reproduction
# Prints 1. Expected 0.
helm template t oci://registry-1.docker.io/temporalio/temporal-worker-controller \
--version 0.33.0 --set metrics.enabled=false | grep -c 'name: kube-rbac-proxy'
Environment/Versions
- Worker Controller v1.14.0, chart 0.33.0
- k3s on ARM64
Additional context
Happy to send a PR to add the sidecar only when metrics are on.
- Lenguaje dominante
- Go
- Estrellas
- 198
- Forks
- 84
- Merge medio
- 2 d 18 h
- PR fusionados (30 d)
- 35
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin 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 temporalio/temporal-worker-controller
-
enhancement
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
temporalio/temporal-worker-controller#692 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
documentation good first issue help wanted
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
temporalio/temporal-worker-controller#691 ·
Los mantenedores suelen responder en 1 día
-
[Feature Request] Configure temporal-worker-controller to use hostNetworkPosiblemente ocupada @josh-ferrell la tomó hace 10 días. Abiertohelm
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
temporalio/temporal-worker-controller#680 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
[Feature Request] Helm chart: support topologySpreadConstraints on the controller DeploymentPosiblemente ocupada @akolacz la tomó hace 17 días. Abiertohelm
temporalio/temporal-worker-controller#658 · 1 reacción · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
rbac
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
temporalio/temporal-worker-controller#654 ·
Los mantenedores suelen responder en 1 día
Todos los issues de temporalio/temporal-worker-controller
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
teradata-labs/loom#574 ·
Los mantenedores suelen responder en 1 día
-
cli: name the CRID when saved link limits fail after publish; soften plain-republish mint warningAbiertoenhancement pr-polish-followup priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
layervai/qurl-integrations#1994 ·
Los mantenedores suelen responder en 1 día
-
[Bug]: `Match exec "step ssh check-host %h"` causes SSH connections through ProxyJump to hangAbiertobug needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
CycloneDX/cyclonedx-go#304 ·
-
[Bug]: snowflake_share fails to read a share attached to a listing with target accounts ('TARGETED ACCOUNTS' in the "to" column)Posiblemente ocupada @akhil9tiet la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
snowflakedb/terraform-provider-snowflake#5366 · 2 comentarios ·
Los mantenedores suelen responder en 1 día