Skip to content

Fix python check#304

Open
lucasmlp wants to merge 1 commit into
ovn-org:branch-25.03from
lucasmlp:branch-25.03
Open

Fix python check#304
lucasmlp wants to merge 1 commit into
ovn-org:branch-25.03from
lucasmlp:branch-25.03

Conversation

@lucasmlp

@lucasmlp lucasmlp commented May 6, 2026

Copy link
Copy Markdown

Relates to #303

@dceara

dceara commented May 7, 2026

Copy link
Copy Markdown
Collaborator

Hi @lucasmlp,

Thanks for the PR. However, we cannot accept it in its current form, it's missing DCO (developer's certificate of origin, i.e., Signed-off-by: ..).

Also, in general, OVN usually expects contributions in the form of patches on the ovs-dev mailing list. We do look at PRs on GitHub occasionally, so not as consistently as we check for patches on the mailing list.

A full description on how to contribute to OVN is here:
https://docs.ovn.org/en/latest/internals/contributing/

For posting patches to the mailing list, this section is the most relevant:
https://docs.ovn.org/en/latest/internals/contributing/submitting-patches.html

If for some reason you have issues sending the fix as a patch email to the dev list please let me know and I can try to assist. In particular for this PR I'd recommend at least updating the commit log with DCO (Signed-off-by: ...) and running checkpatch.py, e.g., on your branch ./utilities/checkpatch.py -1.

Regards,
Dumitru

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants