ci: Test on riscv64 using a self-hosted runner#1059
Conversation
b174918 to
0e95c55
Compare
|
To address the issue. I left some comments on there, please review it also |
|
|
||
| - name: Download musl source | ||
| run: | | ||
| sed -i 's|https://github.com/kraj/musl.git|https://community-ci.openruyi.cn/others/kraj-musl.git|g' ./ci/update-musl.sh |
There was a problem hiding this comment.
I hope these sed usages can be accepted in order to speed up the whole workflow. And this clearly demonstrates what we did.
There was a problem hiding this comment.
Is the speed difference that significant? I'd rather not use the mirrors if possible to reduce the possible points of failure or things that could get out of sync.
There was a problem hiding this comment.
Yes, this is one of the feasible solutions we can implement.
On the self-hosted riscv64 runners, outbound access to some upstream services can be intermittently unstable due to external network factors, which may occasionally affect CI reliability before the actual build or tests start from my experimental. In fact,localized mirror services were included as an initial goal of this project to ensure the stability of CI.
Regarding the risk of mirrors getting out of sync: these mirrors are actively maintained and have a daily synchronization mechanism to track upstream changes, along with basic health checks to ensure they remain usable. This helps reduce the risk of silent divergence.
We do not publish our mirror service in case other users abuse it to affect the stability of our CI, if necessary, we can publish it after our internal assessment.
Using these Prepare mirrored sources with sed is a more transparent approach than using a cached solution, which increases complexity. But for r-l/r, we have to depend on lots of cache strategy, If we get to that point.
There was a problem hiding this comment.
Confirmed with infra on Zulip (starting around here #t-infra > RISC-V runner in compiler builtins @ 💬) that we don't want to use these mirrors. It's fine to keep around while you're still testing, but please drop them at some point.
If there's some weird circumstances making these mirrors work better than GH, bring it up on Zulip so we can see if there's something we can do.
0e95c55 to
b96d13f
Compare
b96d13f to
1a1d2f9
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1a1d2f9 to
3d9410c
Compare
This comment has been minimized.
This comment has been minimized.
c55151b to
ec3f9cb
Compare
This comment has been minimized.
This comment has been minimized.
ec3f9cb to
efe7adb
Compare
This comment has been minimized.
This comment has been minimized.
efe7adb to
91a47f1
Compare
91a47f1 to
9f332b5
Compare
3b5a163 to
b658e1a
Compare
This comment has been minimized.
This comment has been minimized.
|
We can add riscv64 runners first then merge this PR if no problem. |
b658e1a to
7bfad63
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
7bfad63 to
fc7cd1d
Compare
fc7cd1d to
b7f5310
Compare
|
Mind rebasing so we get a fresh run with the more recent CI changes? |
Closes: #1054
ci: test-libm