Hi Han, On 9/17/25 7:09 PM, Han Zhou wrote: > > > On Wed, Sep 17, 2025 at 8:17 AM Numan Siddique <[email protected] > <mailto:[email protected]>> wrote: >> >> On Wed, Sep 17, 2025 at 6:42 AM Dumitru Ceara <[email protected] > <mailto:[email protected]>> wrote: >> > >> > On 9/17/25 11:07 AM, Ales Musil wrote: >> > > On Tue, Sep 16, 2025 at 6:38 PM Mark Michelson via dev < >> > > [email protected] <mailto:[email protected]>> wrote: >> > > >> > >> Signed-off-by: Mark Michelson <[email protected] > <mailto:[email protected]>> >> > >> --- >> > >> .../internals/contributing/submitting-patches.rst | 13 +++++++ > ++++++ >> > >> 1 file changed, 13 insertions(+) >> > >> >> > >> diff --git a/Documentation/internals/contributing/submitting- > patches.rst >> > >> b/Documentation/internals/contributing/submitting-patches.rst >> > >> index c82e49124..1d6fa5e45 100644 >> > >> --- a/Documentation/internals/contributing/submitting-patches.rst >> > >> +++ b/Documentation/internals/contributing/submitting-patches.rst >> > >> @@ -366,6 +366,19 @@ OVN please sign off your work. The sign off >> > >> certifies the following: >> > >> >> > >> See also http://developercertificate.org/ <http:// > developercertificate.org/>. >> > >> >> > >> +AI-assisted Contributions >> > >> +------------------------- >> > >> + >> > >> +OVN is a Linux Foundation project, and the Linux Foundation's > policy for >> > >> +AI-generated patches can be found here: >> > >> +https://www.linuxfoundation.org/legal/generative-ai <https:// > www.linuxfoundation.org/legal/generative-ai> . >> > >> + >> > >> +OVN allows the use of AI assistants in producing patches. > Contributions >> > >> +which have used an AI assistant should disclose the use of the > assistant >> > >> by >> > >> +using the "Assisted-by" tag. While AI-assisted patches are > allowed, the >> > >> author >> > >> +of the patch is ultimately responsible for ensuring that the AI- > generated >> > >> code >> > >> +has not violated any terms of the Developer's Certificate of Origin. >> > >> + > > All looks good to me except that the standard of "Assisted-by" tag is > not very clear. For example, does use of auto-completion count as AI- > assisted? In the past auto-completion was very standard in IDE's but > nowadays it is powered by AI and is much more intelligent and can > generate small/big chunks of code. If so, developers using AI powered > IDE should always add the tag Assisted-by, and it seems not very useful > information. As for myself, I have been exploring Cursor recently and am > in a mixed mode of VIM and Cursor :) >
I would guess it's up to the author of the patch to decide when the AI powered IDE did more than just "type what the author already wanted to type but just quicker". Hence when to add the Asissted-by tag or not. > If the purpose of the tag is to remind reviewers to pay extra attention > to AI-generated code, it may be useful, but we should insist that no > matter if the tag is added or not, the author of the patch is ultimately > responsible for the quality of the code, and as a good practice, she/he > should carefully review/modify the code before submitting the patch. > I would argue that with any patch submission (AI assisted or not) the author should try to ensure the quality of their code is up to standard. So I don't know if we really need to add it here. But I'm not against doing that if you see value in it. > BTW, any suggestion for the value of the Assisted-by tag? For example, > should it be just the IDE (such as Cursor, Windsurf, ChatGPT etc.) or > the actual LLM model being used? Or just Assisted-by: AI? > Here are a couple of different policies from other projects. I've been shown these examples internally: https://github.com/avocado-framework/avocado/blob/master/docs/source/guides/contributor/chapters/ai_policy.rst#transparency-and-disclosure https://github.com/patternfly/.github/blob/main/CONTRIBUTING.md#ai-assisted-development-guidelines Hope that helps. Regards, Dumitru > Best regards, > Han > >> > >> Feature Deprecation Guidelines >> > >> ------------------------------ >> > >> >> >> > >> -- >> > >> >> > > Looks good to me, thanks. I have also added all OVN + few OVS > maintainers >> > > (as it might be useful for ovs too) to cc. >> > > >> > > Acked-by: Ales Musil <[email protected] <mailto:[email protected]>> >> > > >> > >> > Hi Mark, Ales, >> > >> > +Eelco >> > >> > Looks good to me too, thanks! >> > >> > Acked-by: Dumitru Ceara <[email protected] <mailto:[email protected]>> >> >> Looks good to me too, thanks! >> >> Acked-by: Numan Siddique <[email protected] <mailto:[email protected]>> >> >> Numan >> >> > >> > Regards, >> > Dumitru >> > >> > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
