fix(chart): Tweak container command to invoke GarnetServer using abso…

…lute path (#1025) * fix(chart): Tweak container command to invoke GarnetServer using absolute path The relative path seems to be troublesome for NanoServer container for which, for some unclear reasons, executable cannot be found leading to Kubernetes errors like: Error: failed to start containerd task garnet: hcs::System::CreateProcess: ./GarnetServer garnet: The system cannot find the file specified.: unknown despite `WORKDIR` setting in https://github.com/microsoft/garnet/blob/4a26c1280bfd9198c88d21cb7f1eba0a99715c60/Dockerfile.nanoserver#L18 All variations have been tested like`.\\GarnetServer` and with/without `.exe` extension, and only absolute path seems to work. Tests were based on simplest deployment using all the default values. The absolute path also unifies the command across all the containers specs in use i.e. entry point in Dockerfile-s and Helm charet command. Signed-off-by: Mateusz Łoskot <[email protected]> * Run default entrypoint of container image Signed-off-by: Mateusz Łoskot <[email protected]> --------- Signed-off-by: Mateusz Łoskot <[email protected]> Co-authored-by: Tal Zaccai <[email protected]>
microsoft · Feb 28, 2025 · 8f614ca · 8f614ca · github-actions · Feb 28, 2025
1 parent 4b243c4
commit 8f614ca
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/charts/garnet/templates/statefulset.yaml b/charts/garnet/templates/statefulset.yaml
@@ -41,7 +41,6 @@ spec:
             {{- toYaml .Values.securityContext | nindent 12 }}
           image: "{{ .Values.image.registry }}/{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
           imagePullPolicy: {{ .Values.image.pullPolicy }}
-          command: ["./GarnetServer"]
           {{- if or .Values.containers.args .Values.config.garnetConf .Values.config.existingSecret }}
           args:
           {{- range .Values.containers.args }}