-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
CI (Buildkite): Add the package_linux64
and doctest
builders
#41541
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_linux64
builder until thepackage_linux64
builder has completed successfully.package_linux64
builder, upload the build artifacts.tester_linux64
builder, download the build artifacts frompackage_linux64
(instead of building Julia from source).doctest
builder.