Conversation
|
@WardBrian you had a good hunch, seems it doesn't like something in armhf: https://jenkins.mc-stan.org/blue/organizations/jenkins/stanc3/detail/master/934/pipeline/90 |
|
Everything is the same except one thing, the image hash used: For a failure: For a success: Checking ... |
|
So the failing hash is the build variant for ARM from what I understand in https://github.com/stan-dev/stanc3/blob/master/scripts/build_multiarch_stanc3.sh#L23 |
|
Very odd. This is why we test |
|
I've rebuild the image ( on Ubuntu WSL ) but it doesn't have any change, trying now on Mac M1. |
|
No this is new to me. I'm building the armhf image locally now, so will see if there are any differences |
|
Thanks! Quick question, are you on M1 Mac (ARM) CPU or amd64 ? Edit: I've added you as a collaborator to |
|
amd64 for me |
|
Works for me locally, oddly enough. The |
|
I believe we can also remove the |
|
I've tested the others and they work just fine ( also in ci ), but when I pull the specific hash it wants for the armhf build: and then run I get Should we try to override the remote build with your local build?: Building on this M1 chip takes ages :( |
|
Could we set up a github action in the ci-scripts repo to build/push these images? Not sure how involved that would be/how much support GHA has for such a thing. |
|
I had that in mind too, we can set it up but I would suggest using Jenkins as it's a resource-intensive workload. Lmk what you think, I'll create a PR with it in ci-scripts. |
|
It would be great to be able to trigger it without needing anyone to do it on their local machine, even if it's not going to happen that often (probably just now, when #1019 is ready, and then hopefully not for a long time). Definitely not a priority, and if @andrjohns could just push his working one now that would be great for the time being |
|
I don't think you can replace a single architecture when pushing a docker image, would need to rebuild the whole bunch. Did you want me to start that running? |
|
If that wouldn't be too much of a bother @andrjohns many thanks! |
|
Sounds good! Will let you know when it's all rebuilt and pushed |
|
@serban-nicusor-toptal what is |
|
https://jenkins.mc-stan.org/blue/organizations/jenkins/stanc3/detail/master/938/pipeline/86 worked after the latest docker hub push. Thanks @andrjohns ! edit: although it looks like odoc failed. Might just be because I started the CI on an intermediary step? |
Those are used to make win/mac/linux binaries for branches, which is useful for making a test build you can share with others and they dobt need ocaml installed. |
|
@serban-nicusor-toptal I removed |
|
Ah, that is useful @rok-cesnovar. It will need to be updated to use the docker images rather than files, then |
|
Readded and copied over the changes from #1017. We should test after this is merged |
|
Seems like a different one of the non-x86 binaries failed to build. Very weird |
|
Thanks for looking into it! To explain this one here: I also saw this error but it looks like a false positive, is the checkout of the Maybe? I'll check it |
|
@serban-nicusor-toptal is this ready? |
|
I've committed a change that hopefully will fix the random failure with the gh-pages branch. |
|
Because we force-push to gh-pages anyway, if we really needed to I think it's reasonable to make those commands that keep failing something like |
Submission Checklist
Release notes
Move docker to ci-scripts repository.
Removed a comment from
stanc.mlCopyright and Licensing
By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the BSD 3-clause license (https://opensource.org/licenses/BSD-3-Clause)