-
Notifications
You must be signed in to change notification settings - Fork 428
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add ci tests for ir on vineyard #1515
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1515 +/- ##
===========================================
- Coverage 73.61% 40.61% -33.01%
===========================================
Files 88 88
Lines 9681 9681
===========================================
- Hits 7127 3932 -3195
- Misses 2554 5749 +3195
Continue to review full report at Codecov.
|
@@ -65,6 +65,11 @@ jobs: | |||
cd ${GITHUB_WORKSPACE}/research/query_service/ir/compiler | |||
make build | |||
|
|||
- name: Build Ir on Vineyard Store | |||
run: | | |||
source ${HOME}/.bashrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to source this bashrc? It is user's customed envs.
If you have envs, should put them together into the .graphscope_env
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why, but we do have many source ~/.bashrc
lines across all CI yamls. It might be used for leveraging local build cache or something like that, I guess.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments.
@@ -86,3 +91,9 @@ jobs: | |||
source ${HOME}/.bashrc | |||
cd ${GITHUB_WORKSPACE}/interactive_engine/ir-adaptor && ./ir_groot_ci.sh | |||
|
|||
- name: Ir Integration Test on Vineyard Store | |||
run: | | |||
source ${HOME}/.bashrc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Related issue number
Fixes