You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/01/11 08:05:50 UTC
[kyuubi] branch master updated: [KYUUBI #4034][BUILD][FOLLOWUP] Update build script due to helm charts change
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 <22...@qq.com>
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 <22...@qq.com>
Signed-off-by: Cheng Pan <ch...@apache.org>
---
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"