(kyuubi) branch branch-1.9 updated: [KYUUBI #6430] livenessProbe uses absolute path based on KYUUBI_HOME

[ch...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

chengpan pushed a commit to branch branch-1.9
in repository https://gitbox.apache.org/repos/asf/kyuubi.git


The following commit(s) were added to refs/heads/branch-1.9 by this push:
     new eed7f4209 [KYUUBI #6430] livenessProbe uses absolute path based on KYUUBI_HOME
eed7f4209 is described below

commit eed7f42090e5ec3ff8f8d84baad458093196e252
Author: camper42 <ca...@gmail.com>
AuthorDate: Thu May 30 12:11:25 2024 +0800

    [KYUUBI #6430] livenessProbe uses absolute path based on KYUUBI_HOME
    
    # :mag: Description
    ## Issue References 🔗
    
    This pull request fixes potential `Liveness probe failed: /bin/bash: line 1: bin/kyuubi: No such file or directory`
    
    ## Describe Your Solution 🔧
    
    livenessProbe uses absolute path based on KYUUBI_HOME
    
    ## Types of changes :bookmark:
    
    - [x] Bugfix (non-breaking change which fixes an issue)
    - [ ] New feature (non-breaking change which adds functionality)
    - [ ] Breaking change (fix or feature that would cause existing functionality to change)
    
    ## Test Plan 🧪
    
    Tested on our internal kyuubi deployment.
    
    ---
    
    # Checklist 📝
    
    - [x] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)
    
    **Be nice. Be informative.**
    
    Closes #6430 from camper42/liveness.
    
    Closes #6430
    
    980b42cdd [camper42] livenessProbe uses absolute path based on KYUUBI_HOME
    
    Authored-by: camper42 <ca...@gmail.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
    (cherry picked from commit 9c4e832395cf211a8dba6cc472842356914c75cb)
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 charts/kyuubi/templates/kyuubi-statefulset.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/charts/kyuubi/templates/kyuubi-statefulset.yaml b/charts/kyuubi/templates/kyuubi-statefulset.yaml
index a79b5be9a..303103fdd 100644
--- a/charts/kyuubi/templates/kyuubi-statefulset.yaml
+++ b/charts/kyuubi/templates/kyuubi-statefulset.yaml
@@ -87,7 +87,7 @@ spec:
           {{- if .Values.livenessProbe.enabled }}
           livenessProbe:
             exec:
-              command: ["/bin/bash", "-c", "bin/kyuubi status"]
+              command: ["/bin/bash", "-c", "$KYUUBI_HOME/bin/kyuubi status"]
             initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds }}
             periodSeconds: {{ .Values.livenessProbe.periodSeconds }}
             timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds }}