ArroyoSystems / arroyo

Distributed stream processing engine in Rust
https://arroyo.dev
Apache License 2.0
3.45k stars 189 forks source link

Fix a bug in the k8s deployment where the worker. slots configuration… #595

Closed FourSpaces closed 2 months ago

FourSpaces commented 2 months ago

When I was deploying using k8s, I found that there seemed to be a problem with the configuration of the K8S_WORKER_SLOTS environment variable, and I fixed it to the correct value.

https://github.com/ArroyoSystems/arroyo/blob/5fc6fe06cbbdc866f232bd813eb5e8aff16bcb3a/k8s/arroyo/templates/controller.yaml#L104C1-L105C51

        - name: K8S_WORKER_SLOTS
          value: {{ .Values.workerSlots | quote }}

.Values.workerSlots does not exist, .Values.worker.slots exists, so I modified it to .Values.worker.slots

FourSpaces commented 2 months ago

@mwylde Please help me review