search

paritytech / scripts

Collection of Dockerfiles, scripts and related resources
Apache License 2.0
41 stars 39 forks source link

Use `clang-14` for `paritytech/base-ci-linux` #488

Closed rcny closed 1 year ago

rcny commented 1 year ago

We have linker failures in the Substrate CI most possibly related that clang-13 is too old for currently used Rust versions. This PR bumps clang from 13 to 14 for the base image and the dependent ci-linux image.

altaua commented 1 year ago

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

altaua commented 1 year ago

Oh I see the *_VERSION flags aren't being passed in for some reason.

alvicsam commented 1 year ago

About CI - I added the jobs that work in PR recently, so there might be some bugs

muharem commented 1 year ago

@alvicsam nether it does not help, or my jobs running on the prev ci script version https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2081682

samelamin commented 1 year ago

Same thing for me

https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2081326

rcny commented 1 year ago

We are unable to release the updated images due to the same linker error. Need more time to investigate, clang-14 didn't help.