-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
config: pipeline: Update bookworm-rt rootfs #700
config: pipeline: Update bookworm-rt rootfs #700
Conversation
I've set the rootfs URL to staging. After verification I'll put URL of production before merging on this PR. |
c87a215
to
8d7dd5b
Compare
Sounds good, can you please set this PR as draft so it can't get merged as-is by accident? |
3f56ab2
to
90d6698
Compare
LAVA job referenced in link is from production, which is not updated yet |
The test failed in reality as
From above node link, I've found the grand parent node (https://staging.kernelci.org:9000/viewer?node_id=66bf5f908ac59a075e697801) to find lava log. This log shows that older and wrong kselftest rootfs is being used:
Why the bookworm-rt isn't being used here? @nuclearcat @pawiecz please can you help debug this? |
@musamaanjum It looks like only depthcharge boot method currently supports |
Many thanks @pawiecz. I was scratching my head over it from some time. @nuclearcat @a-wai can we schedule the work to update the templates? You may have pre-planned it already. We should prioritize fixing rt-tests work as the work started on the rt maintainer's request and we don't want to lose his interest. (cc: @helen-fornazier as she is leading the community engagement work and can help in deciding the priority of this work.) |
@musamaanjum this was trivial enough that I pushed kernelci/kernelci-core#2641 |
Setting staging-skip to allow testing #766 |
#766 has been merged, dropping |
Note: with #766 merged it now conflicts, so this also needs a rebase to be deployed. |
New package has been added to bookworm-rt rootfs. Hence update the URL to newly updated rootfs. Signed-off-by: Muhammad Usama Anjum <[email protected]>
90d6698
to
28d74d5
Compare
@a-wai Thanks for fixing. Rebased the PR. |
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.
LGTM
Most recently triggered Maestro Jobs (bottom) showed that the LAVA TestJob definition might still need some adjustments but the rootfs now contains necessary tools which should be enough to merge this PR as well as kernelci/kernelci-core#2614
I've collected the results on kernelci/kernelci-project#439 and fix issues under this ticket. Overall, 3/11 tests have issues in their executor script in https://github.com/Linaro/test-definitions. |
New package has been added to bookworm-rt rootfs. Hence update the URL to newly updated rootfs.
Related PR: kernelci/kernelci-core#2614