-
Notifications
You must be signed in to change notification settings - Fork 29
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
kernelci.org: add blog post about the Patchwork Integration #296
base: main
Are you sure you want to change the base?
Conversation
Publish blog post about the under-development Patchwork Integration. Signed-off-by: Gustavo Padovan <[email protected]>
|
||
So recently, Nikolay Yurin, a Production Engineer from Meta, started developing the support [patchwork in the KernelCI](https://github.com/getpatchwork/patchwork/issues/557). It is still under heavy development and should reach a point that we can experiment with in early 2024. | ||
|
||
It is very important to put experimental support in place as we still don't know all the challenges we will face with the Patchwork integration, so engaging with the community to gather early feedback will be quite helpful. There are definitely corner cases and considerations that we will find out only when we start experimenting with it. |
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.
started developing the support
I'd rather say suggested a way to implement :) So far I did nothing on patchwork side. However it would worth mentioning this kernelci/kernelci-api#307 and maybe kernelci/kernelci-pipeline#342
linkTitle: "Adding Patchwork integration to KernelCI" | ||
author: Gustavo Padovan | ||
description: > | ||
Upcoming Patchwork integration in KernelCI aims helping maintainers reduce their workload, by deferring first line level of checks to KernelCI. |
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.
Hi Gustavo,
Thank you for the MR.
Minor suggestion:
Upcoming Patchwork integration in KernelCI aims helping maintainers reduce their workload, by deferring first line level of checks to KernelCI. | |
Upcoming Patchwork integration in KernelCI aims to help maintainers reduce their workload, by deferring the first level of checks to KernelCI. |
linkTitle: "Adding Patchwork integration to KernelCI" | ||
author: Gustavo Padovan | ||
description: > | ||
Upcoming Patchwork integration in KernelCI aims helping maintainers reduce their workload, by deferring first line level of checks to KernelCI. |
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.
Upcoming Patchwork integration in KernelCI aims helping maintainers reduce their workload, by deferring first line level of checks to KernelCI. | |
Upcoming Patchwork integration in KernelCI aims to help maintainers reduce their workload by deferring first line level of checks to KernelCI. |
|
||
Last month, at [Linux Plumbers](https://lpc.events/), most of the KernelCI Board Members were present and engaging with the community on several kernel testing related topics. One topic in particular has been grabbing the attention of some kernel maintainers: the ability to test patches out of Patchwork automatically. That way, maintainers can have a round of automated testing done before they put their hands on the patches. | ||
|
||
Testing the patches at the moment they arrive at the mailing list may be one of the best approaches because we can identify issues even before a person puts their eyes on the code. Developing a robust system to test new patches landing in Patchwork is not a trivial task. It will take some time to stabilize the support and deal with different corner cases. There is also the resource availability side, as doing that kind of testing at scale for several subsystems requires a lot of compute power. Last but not least, security is key, as blindly testing any patch that lands on the mailing list is not a great idea security-wise. |
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.
Style suggestion: The previous line uses "put their hands on the patches", this one uses "puts their eyes on the code". Find an alternative phrasing for the latter.
|
||
The big advantage of adding such support in KernelCI is that we can solve the problem for virtually all subsystems at the same time without reinventing and maintaining different CI infra across the board. | ||
|
||
So recently, Nikolay Yurin, a Production Engineer from Meta, started developing the support [patchwork in the KernelCI](https://github.com/getpatchwork/patchwork/issues/557). It is still under heavy development and should reach a point that we can experiment with in early 2024. |
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.
So recently, Nikolay Yurin, a Production Engineer from Meta, started developing the support [patchwork in the KernelCI](https://github.com/getpatchwork/patchwork/issues/557). It is still under heavy development and should reach a point that we can experiment with in early 2024. | |
So recently, Nikolay Yurin, a Production Engineer from Meta, started developing support for [patchwork in KernelCI](https://github.com/getpatchwork/patchwork/issues/557). It is still under heavy development and should reach a point that we can experiment with in early 2024. |
Mentioning Patchwork is kind of a "buzzword", it can attract some readers as it's well known but it would be fair to say that this feature is really about pre-merge testing in general and Patchwork is just one way of doing it on the client side. |
Beyond a buzzword, it is the main way many kernel maintainers do their work. So telling the community that we want to serve that, whet some maintainers are already interested in working with us on that is important in my opinion. Also, I agree this could eventually be a generic pre-merge testing feature, but right now there is only interest and effort available for the Patchwork part. And I believe it is in best interest of the project to make that integration happen. |
Yes, I mean it would seem useful to make it clearer that the API is not going to be tied to Patchwork. It's only a starting point with the BPF use-case. |
Let me add a note about that in the post then. I think it is important to highlight that we want to solve pre-merge in general too. |
@padovan Hello, is this still being worked on or has the blog post idea been abandoned for now? |
We are still working on it, but I believe we have to wait a bit more to publish it. |
Publish blog post about the under-development Patchwork Integration, so we can raise the awareness of the work and get people to follow it from now on.