Executor Helm Chart - kubernetesJob.node.tolerations doesn't render as expected

Abierto
#349 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
55/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Estancado
Stack tecnológico
helm, kubernetes
Área
devops, infrastructure

Línea de trabajo

Inspecciona charts/sourcegraph-executor/k8s/templates/executor.ConfigMap.yaml en la línea 26 y el valor de tolerations en charts/sourcegraph-executor/k8s/values.yaml alrededor de las líneas 116-117. Renderiza el chart con el valor documentado de tolerations y compara la salida de la ConfigMap con el JSON que espera el executor. Se considera terminado cuando el valor de EXECUTOR_KUBERNETES_NODE_TOLERATIONS sigue siendo JSON válido y el executor puede analizarlo.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

When setting the value for executor.kubernetesJob.node.tolerations it does not render in the configmap in the way that the executor pod is able to consume it.

I'm finding that if you set the following as the chart suggests:
tolerations: [{"key": "foo", "operator": "Equal", "value": "bar", "effect": "NoSchedule"}]
It will render as EXECUTOR_KUBERNETES_NODE_TOLERATIONS: map[effect:NoSchedule key:foo operator:Equal value:bar]
Causing your executor pod to throw invalid EXECUTOR_KUBERNETES_NODE_TOLERATIONS, failed to parse: invalid character 'm' looking for beginning of value into the logs and does not execute jobs.

If you do a manual edit of the configmap back to the original Json specified in the chart, then the jobs executor and respect the tolerations.

https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph-executor/k8s/values.yaml#L116-L117

I think the chart is incorrectly trying to render the json to yaml, when it should remain as json.
https://github.com/sourcegraph/deploy-sourcegraph-helm/blob/main/charts/sourcegraph-executor/k8s/templates/executor.ConfigMap.yaml#L26

Lenguaje dominante
Go Template
Estrellas
19
Forks
24
Merge medio
1 d 4 h
PR fusionados (30 d)
13

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de sourcegraph/deploy-sourcegraph-helm

Todos los issues de sourcegraph/deploy-sourcegraph-helm

Issues similares

Más issues de DevOps

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.