-
Notifications
You must be signed in to change notification settings - Fork 65
[ML] Run QA tests on PR builds #2592
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
Conversation
Add the option to run a subset of QA tests against a locally built distribution of elasticsearch + ml-cpp
| - 'buildkite-agent artifact download "build/*" . --step build_test_linux-aarch64-RelWithDebInfo' | ||
| - '.buildkite/scripts/steps/run_qa_tests.sh || (cd ../elasticsearch && find x-pack -name logs | xargs tar cvzf logs.tgz && buildkite-agent artifact upload logs.tgz && false)' | ||
| depends_on: "build_test_linux-x86_64-RelWithDebInfo" |
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 download the build artifacts for ARM but depend on Intel?
If we only run this on one platform I'd prefer it was Intel, because that's what we've chosen to run on in our production Cloud.
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 download the build artifacts for ARM but depend on Intel?
Yeah, that's a typo, I intended it to be intel for that very reason. Plus exercising the IPEX library in due course.
|
buildkite test this please |
droberts195
left a comment
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.
LGTM
|
buildkite run_qa_tests on linux |
|
buildkite run_qa_tests |
|
buildkite run_qa_tests |
|
buildkite run_qa_tests |
Add the option to run a subset of QA tests against a locally built distribution of elasticsearch + ml-cpp