-
Notifications
You must be signed in to change notification settings - Fork 39
Use clang-14
for paritytech/base-ci-linux
#488
Conversation
Hmm, several ci jobs had just failed due to transient network failures and I approved this pr before they'd all finished running again, but now it turns out several of them are actually legit failing, looks like some downloaded files are getting truncated |
Oh I see the *_VERSION flags aren't being passed in for some reason. |
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.
approving again, aiui now this is going to work when merged, it's just the *-pr jobs that don't set the *_VERSION variables
About CI - I added the jobs that work in PR recently, so there might be some bugs |
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.
we need this (:
@alvicsam nether it does not help, or my jobs running on the prev ci script version |
Same thing for me https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2081326 |
We are unable to release the updated images due to the same linker error. Need more time to investigate, |
We have linker failures in the Substrate CI most possibly related that
clang-13
is too old for currently used Rust versions. This PR bumpsclang
from13
to14
for the base image and the dependentci-linux
image.