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

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


The following commit(s) were added to refs/heads/master by this push:
     new 935864639 [KYUUBI #4034][BUILD][FOLLOWUP] Update build script due to 
helm charts change
935864639 is described below

commit 935864639850e3f3ffa67b1cb30c1c4b6b655d4a
Author: zwangsheng <[email protected]>
AuthorDate: Wed Jan 11 16:05:40 2023 +0800

    [KYUUBI #4034][BUILD][FOLLOWUP] Update build script due to helm charts 
change
    
    ### _Why are the changes needed?_
    
    After changes in #4034, we should modify `./build/dist` to build kyuubi 
binary with charts
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including 
negative and positive cases if possible
    
    - [x] Add screenshots for manual tests if appropriate
    
    - [ ] [Run 
test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    
![WX20230111-155842](https://user-images.githubusercontent.com/52876270/211750105-32ede463-ca6d-41ff-aea4-fd19a200bf8c.png)
    
    Closes #4143 from zwangsheng/helm/build_dist_with_charts.
    
    Closes #4034
    
    78edac4e [zwangsheng] [BUILD] Should build binary with charts dir
    
    Authored-by: zwangsheng <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 build/dist | 1 +
 1 file changed, 1 insertion(+)

diff --git a/build/dist b/build/dist
index 062bc49bc..de18f59c1 100755
--- a/build/dist
+++ b/build/dist
@@ -349,6 +349,7 @@ fi
 cp -r "$KYUUBI_HOME/bin" "$DISTDIR"
 cp -r "$KYUUBI_HOME/conf" "$DISTDIR"
 cp -r "$KYUUBI_HOME/docker" "$DISTDIR"
+cp -r "$KYUUBI_HOME/charts" "$DISTDIR"
 
 if [[ "$MAKE_TGZ" == "true" ]]; then
   TARDIR_NAME="apache-kyuubi-$VERSION-bin$SUFFIX"

Reply via email to