Re: Patchtest results for [OE-core][PATCH v2 1/5] cve-check: annotate CVEs during analysis

[Alexandre Belloni <alexandre.belloni@bootlin.com>]

On 24/07/2024 06:31:47+0200, Marta Rybczynska wrote:
> Ah.. it requires my SoB when the patch is from someone else? That could
> explain the situation.


Patches you submit on behalf of someone else need you SoB , see (c) of
the DCO:

https://developercertificate.org/

> 
> Still, it reported an error to patch 1/5 while the issue is in 2/5. Will
> check what we can improve in the tool...
> 
> Kind regards,
> Marta
> 
> On Wed, Jul 24, 2024 at 1:28 AM Alexandre Belloni <
> alexandre.belloni@bootlin.com> wrote:
> 
> > On 23/07/2024 17:41:40+0200, Marta Rybczynska wrote:
> > > This seems to be a false positive. Possibly the path 2/5 has an issue.
> > I'm
> > > looking into the source code of patchtest to verify.
> >
> > It is not, 2/5 is missing your SoB
> >
> > >
> > > Regards,
> > > Marta
> > >
> > > On Tue, Jul 23, 2024 at 2:12 PM <patchtest@automation.yoctoproject.org>
> > > wrote:
> > >
> > > > Thank you for your submission. Patchtest identified one
> > > > or more issues with the patch. Please see the log below for
> > > > more information:
> > > >
> > > > ---
> > > > Testing patch
> > > >
> > /home/patchtest/share/mboxes/v2-1-5-cve-check-annotate-CVEs-during-analysis.patch
> > > >
> > > > FAIL: test Signed-off-by presence: Mbox is missing Signed-off-by. Add
> > it
> > > > manually or with "git commit --amend -s"
> > > > (test_mbox.TestMbox.test_signed_off_by_presence)
> > > >
> > > > PASS: pretest pylint (test_python_pylint.PyLint.pretest_pylint)
> > > > PASS: test author valid (test_mbox.TestMbox.test_author_valid)
> > > > PASS: test commit message presence
> > > > (test_mbox.TestMbox.test_commit_message_presence)
> > > > PASS: test max line length
> > > > (test_metadata.TestMetadata.test_max_line_length)
> > > > PASS: test mbox format (test_mbox.TestMbox.test_mbox_format)
> > > > PASS: test non-AUH upgrade (test_mbox.TestMbox.test_non_auh_upgrade)
> > > > PASS: test pylint (test_python_pylint.PyLint.test_pylint)
> > > > PASS: test shortlog format (test_mbox.TestMbox.test_shortlog_format)
> > > > PASS: test shortlog length (test_mbox.TestMbox.test_shortlog_length)
> > > >
> > > > SKIP: pretest src uri left files: No modified recipes, skipping pretest
> > > > (test_metadata.TestMetadata.pretest_src_uri_left_files)
> > > > SKIP: test CVE check ignore: No modified recipes or older target
> > branch,
> > > > skipping test (test_metadata.TestMetadata.test_cve_check_ignore)
> > > > SKIP: test CVE tag format: No new CVE patches introduced
> > > > (test_patch.TestPatch.test_cve_tag_format)
> > > > SKIP: test Signed-off-by presence: No new CVE patches introduced
> > > > (test_patch.TestPatch.test_signed_off_by_presence)
> > > > SKIP: test Upstream-Status presence: No new CVE patches introduced
> > > > (test_patch.TestPatch.test_upstream_status_presence_format)
> > > > SKIP: test bugzilla entry format: No bug ID found
> > > > (test_mbox.TestMbox.test_bugzilla_entry_format)
> > > > SKIP: test lic files chksum modified not mentioned: No modified
> > recipes,
> > > > skipping test
> > > >
> > (test_metadata.TestMetadata.test_lic_files_chksum_modified_not_mentioned)
> > > > SKIP: test lic files chksum presence: No added recipes, skipping test
> > > > (test_metadata.TestMetadata.test_lic_files_chksum_presence)
> > > > SKIP: test license presence: No added recipes, skipping test
> > > > (test_metadata.TestMetadata.test_license_presence)
> > > > SKIP: test series merge on head: Merge test is disabled for now
> > > > (test_mbox.TestMbox.test_series_merge_on_head)
> > > > SKIP: test src uri left files: No modified recipes, skipping pretest
> > > > (test_metadata.TestMetadata.test_src_uri_left_files)
> > > > SKIP: test summary presence: No added recipes, skipping test
> > > > (test_metadata.TestMetadata.test_summary_presence)
> > > > SKIP: test target mailing list: Series merged, no reason to check other
> > > > mailing lists (test_mbox.TestMbox.test_target_mailing_list)
> > > >
> > > > ---
> > > >
> > > > Please address the issues identified and
> > > > submit a new revision of the patch, or alternatively, reply to this
> > > > email with an explanation of why the patch should be accepted. If you
> > > > believe these results are due to an error in patchtest, please submit a
> > > > bug at https://bugzilla.yoctoproject.org/ (use the 'Patchtest'
> > category
> > > > under 'Yocto Project Subprojects'). For more information on specific
> > > > failures, see: https://wiki.yoctoproject.org/wiki/Patchtest. Thank
> > > > you!
> > > >
> >
> > >
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > > Links: You receive all messages sent to this group.
> > > View/Reply Online (#202408):
> > https://lists.openembedded.org/g/openembedded-core/message/202408
> > > Mute This Topic: https://lists.openembedded.org/mt/107503127/3617179
> > > Group Owner: openembedded-core+owner@lists.openembedded.org
> > > Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [
> > alexandre.belloni@bootlin.com]
> > > -=-=-=-=-=-=-=-=-=-=-=-
> > >
> >
> >
> > --
> > Alexandre Belloni, co-owner and COO, Bootlin
> > Embedded Linux and Kernel engineering
> > https://bootlin.com
> >

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com