CI Improvements: Add RTAI build#3983
Conversation
|
@BsAtHome I already started as discussed in #3978 and it seams to do something. Seams my first try already fails as it should: https://github.com/hdiethelm/linuxcnc-fork/actions/runs/25117897747/job/73610085493 Next try is with your fix cherry-picked, so I can test if it passes now. |
|
@BsAtHome |
|
@NTULINUX |
|
This is an improvement! Besides running into some performance issues that are of github's own making... They often stall on getting packages. You mentioned a more mature installed docker image. Do you know how that works? |
|
|
BTW, I killed the CI run after I saw the failure (only because I am at the computer...). No need to run the other stuff while there are errors. It will only delay the new test and unnecessary use resources. |
Let's put it that way: I know how to do it in Gitlab. And it is sure possible in Github. What you do basically:
The only issue I see is that when a new dependency is added, the image has to be updated somehow. But a hybrid approach could be used:
The images can be generated directly from the linuxcnc repo or also from an other repo. Normally, you have something like a linuxcnc-build-image repo which does this, so you can push tags there to trigger a build. |
I forgot one line, I have to try again. This CI stuff is always annoying, commit&push until it works. Just tell me if it gets to much and I can try to create my own runners. |
|
Now it works again. This does the job already for RTAI. For future improvements, I would need to know a few details about how the LinuxCNC github is set up.
If you don't feel like sharing this here, you contact me by mail. If you don't feel like sharing this at all, I can create a setup that works in my gitlab and share it. Internals are often a bit sensitive because they can be exploited sometimes. Or I can also give you hints how to do it and review your changes. What I found so far: |
Add RTAI build to CI so RTAI is not broken all the time
|
After #3981 got merged, I re-based on top of master and squashed the test commits. I will mark this as ready, feel free to merge so RTAI is not broken again after the next merge. As already stated, I am ready to help improving the CI even more. |
|
@andypugh, do you know what the deal is with github and CI usage and storage limits? There are some open questions from #3983 (comment) and you probably know more about potential (historic) deals than anybody else. Anyhooo, this PR should be merged, I think, because it addresses the long standing issue of continual RTAI breakage. |
I've made a lot of improvements since then, I would recommend using 5.4.302 from here: |
I switched to these packages and left the old variant commented out if someone needs it one day. |
|
Awesome, thanks! |
In this PR, I will try to improve the CI
There will be a few pushes, including some that should fail on purpose to test if everything works as desired. Just tell me if I abuse the CI to much and I will find a different solution.
Edit: I think it makes sense to merge this and continue in a separate PR so RTAI is not broken again in a few weeks.