Skip to content

Commit

Permalink
fix(github): fix redundant hadoop-bin while uploading artifact
Browse files Browse the repository at this point in the history
  • Loading branch information
empiredan committed Sep 29, 2024
1 parent 73d4bf7 commit 7407881
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 3 deletions.
2 changes: 2 additions & 0 deletions .github/actions/rebuild_thirdparty_if_needed/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,4 +46,6 @@ runs:
../admin_tools/download_zk.sh zookeeper-bin
rm -rf hadoop-bin/share/doc
rm -rf zookeeper-bin/docs
mv hadoop-bin ..
mv zookeeper-bin ..
shell: bash
2 changes: 2 additions & 0 deletions .github/actions/unpack_prebuilt_thirdparties/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,6 @@ runs:
find ./thirdparty -name '*CMakeFiles*' -type d -exec rm -rf "{}" +
rm -rf ./thirdparty/hadoop-bin/share/doc
rm -rf ./thirdparty/zookeeper-bin/docs
mv hadoop-bin ..
mv zookeeper-bin ..
shell: bash
2 changes: 0 additions & 2 deletions .github/actions/upload_artifact/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ runs:
steps:
- name: Tar files
run: |
mv thirdparty/hadoop-bin ./
mv thirdparty/zookeeper-bin ./
rm -rf thirdparty
# The following operations are tricky, these directories and files don't exist if not build with '--test'.
# When build binaries for client tests, it's not needed to add '--test'.
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/lint_and_test_cpp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ on:
- ci-test # testing branch for github action
- '*dev'
paths:
- .github/actions
- .github/actions/**
- .github/workflows/lint_and_test_cpp.yaml
- .github/workflows/thirdparty-regular-push.yml
- build_tools/pack_server.sh
Expand Down

0 comments on commit 7407881

Please sign in to comment.