Skip to content
This repository was archived by the owner on Nov 4, 2024. It is now read-only.

Use clang-15 for paritytech/base-ci-linux#490

Merged
rcny merged 1 commit intomasterfrom
vi-clang-15
Nov 24, 2022
Merged

Use clang-15 for paritytech/base-ci-linux#490
rcny merged 1 commit intomasterfrom
vi-clang-15

Conversation

@rcny
Copy link
Contributor

@rcny rcny commented Nov 24, 2022

clang-14 is still too low for the CI workloads even there were reports that it fixes related linker problems locally. Bumping further to clang-15.

cc @muharem @samelamin

@rcny rcny requested a review from a team November 24, 2022 14:06
@rcny rcny self-assigned this Nov 24, 2022
@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

Supersedes #488.

@rcny rcny merged commit c27cc84 into master Nov 24, 2022
@rcny rcny deleted the vi-clang-15 branch November 24, 2022 14:08
@samelamin
Copy link

Is it safe to rerun workloads now?

@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

Not yet, we're rebuilding the images, I'll post an update here when the we'll push updated images.

@rcny
Copy link
Contributor Author

rcny commented Nov 24, 2022

So, looks like it's not related to the clang version as we continue to observe the same behavior with the same linker error even with clang-15. I will investigate more.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants