-
Notifications
You must be signed in to change notification settings - Fork 110
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
new(driverkit/config): add build config for 6.1.0-rpi7-rpi-2712_1 #1323
Conversation
Welcome @Nabsku! It looks like this is your first PR to falcosecurity/test-infra 🎉 |
Hi @Nabsku. Thanks for your PR. I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: Nabsku The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Nabsku <[email protected]>
45637a0
to
58b7e08
Compare
/ok-to-test |
Hi! Thanks for opening this PR! |
Of course! It's a RaspiOS Debian running on a Pi5.
|
Thanks! I am trying to understand why we are not catching any We can try merging this one but i fear that without specifying EDIT: unfortunately my PR on kernel-crawler is not enough :/ falcosecurity/kernel-crawler#192 (comment) |
Am not the most proficient kernel guru but I can try and find these kernelurls, if that helps? But it seems we've uncovered a bug in the crawler then? |
Exactly :)
Yes that would be great! If you check other debian configs in the same folders, you'll see how the kernelurls are added! |
Signed-off-by: Nabsku <[email protected]>
I believe/hope this is correct. Now that I think of it, does the crawler look for kernels in the Raspberry repos? |
driverkit/config/4.0.0+driver/aarch64/debian_6.1.0-rpi7-rpi-2712_1.yaml
Outdated
Show resolved
Hide resolved
I don't think so, that's the problem :D EDIT: i am trying locally to add |
Signed-off-by: Nabsku <[email protected]>
Yay:
|
Awesome! |
If we are quick enough to merge the PR, i would let the next kernel-crawler pipeline run (tomorrow morning) create the new driverkit configs for us! Wdyt? |
I'm all for it :) Is there anything left to do? If so just say the word! |
Nothing, as soon as my kenrel-crawler patch gets merged, i will tag a new release and tomorrow morning you will find Falco drivers built and published on |
Shall I close this PR? Probably not needed anymore, right? :) Thanks for your help! |
I think so! You are welcome, thanks for spotting the issue! |
Oh I didn't notice the "1:" prefix in the output kernel release:
Will need to fix this on Monday; sorry! |
Ok fixed, and currently re running kernel-crawler: https://github.com/falcosecurity/kernel-crawler/actions/runs/7246238172. |
Unfortunately, it seems driverkit is not able to build the drivers for rpi debian kernels. |
We finally have prebuilt drivers for |
No description provided.