sourcegraph / sourcegraph/deploy-sourcegraph-helm

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

Offen
#349 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Go Template
Sterne
18
Forks
24
Ø Merge
1 T. 4 Std.
Gemergte PRs (30 T.)
13

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Untersuche charts/sourcegraph-executor/k8s/templates/executor.ConfigMap.yaml in Zeile 26 und den Wert für tolerations in charts/sourcegraph-executor/k8s/values.yaml etwa in den Zeilen 116–117. Rendere das Chart mit dem dokumentierten tolerations-Wert und vergleiche die ConfigMap-Ausgabe mit dem JSON, das der Executor erwartet. Als erledigt gilt die Aufgabe, wenn der Wert von EXECUTOR_KUBERNETES_NODE_TOLERATIONS weiterhin gültiges JSON ist und der Executor ihn parsen kann.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
helm, kubernetes
Bereich
devops, infrastructure
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.