-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
CI (Buildkite): Add the package_linux64 and doctest builders
#41541
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
bc672eb to
fe9e16a
Compare
package_linux64 and tester_linux64 pipelinespackage_linux64 and tester_linux64 builders
fe9e16a to
6687f26
Compare
package_linux64 and tester_linux64 builderspackage_linux64, tester_linux64, and doctest builders
04af6c8 to
e8721ce
Compare
@staticfloat Take a look at the test failures. It looks like most are due to issues with the sandboxing? |
cae3c03 to
4b07254
Compare
|
Wow, this is a lot of work Dilum! Awesome to have you working on these pieces. A few things to note:
A quick glance over the rest of the errors all look like they stem from |
I wouldn't get your hopes up 😂. I'm thinking that for this PR, we focus on getting everything working without |
4b07254 to
f7e9cb2
Compare
080d98a to
6cf3c28
Compare
|
Okay, looks like we are making progress. Originally, we had 12 failures and 6 errors, and now we are down to 10 failures and 0 errors. |
|
@Keno We're running Julia tests inside of |
|
Seems inconsistent, but that's Linux for you. I'd just allow EINVAL too in that test. |
|
I was thinking that it would be nice to keep this PR as Buildkite-changes-only. What do y'all think of making a separate PR to modify the tests, and then rebasing this PR once the test-modifying-PR is merged? |
#41682 (I also made a few changes in that PR to account for the fact that |
6b3d403 to
cbd7e10
Compare
|
Any idea why these |
590ca10 to
a97b391
Compare
|
These
It's kind of strange that the process |
|
Here's an update: the My plan is:
|
package_linux64, tester_linux64, and doctest builderspackage_linux64 and doctest builders
Co-authored-by: Elliot Saba <[email protected]>
7be53da to
eb05df6
Compare
|
I took all the TODOs that we did not complete in this PR, and I moved them to #41509. |
) Co-authored-by: Elliot Saba <[email protected]> (cherry picked from commit c7097c5)
) Co-authored-by: Elliot Saba <[email protected]> (cherry picked from commit c7097c5)
) Co-authored-by: Elliot Saba <[email protected]> (cherry picked from commit c7097c5)
) Co-authored-by: Elliot Saba <[email protected]> (cherry picked from commit c7097c5)
) Co-authored-by: Elliot Saba <[email protected]> (cherry picked from commit c7097c5)
TODO:
tester_linux64builder until thepackage_linux64builder has completed successfully.package_linux64builder, upload the build artifacts.tester_linux64builder, download the build artifacts frompackage_linux64(instead of building Julia from source).doctestbuilder.