On 20 Sep 2024, at 11:51, Ilya Maximets wrote:
> On 9/16/24 12:32, Eelco Chaudron wrote:
>> The oss-fuzz project builds specific OVS fuzzing code located in the
>> tests/oss-fuzz/ directory of our repository. However, this code is
>> not currently built as part of our CI pipeline, creating a potential
>> risk that changes in the main OVS code could break the oss-fuzz
>> integration. This commit addresses that by ensuring the fuzzing code
>> is built during CI, preventing potential issues. The additional build
>> step takes approximately 4 minutes.
>>
>> Signed-off-by: Eelco Chaudron <[email protected]>
>> ---
>> .github/workflows/build-and-test.yml | 26 ++++++++++++++++++++++++++
>> 1 file changed, 26 insertions(+)
>>
>> diff --git a/.github/workflows/build-and-test.yml
>> b/.github/workflows/build-and-test.yml
>> index 9d3a13ca1..5dd0cc2ad 100644
>> --- a/.github/workflows/build-and-test.yml
>> +++ b/.github/workflows/build-and-test.yml
>> @@ -388,6 +388,32 @@ jobs:
>> - name: build
>> run: ./.ci/linux-build.sh
>>
>> + build-oss-fuzz:
>> + name: Build oss-fuzz fuzzers
>
> I'd change the 'Build' to lowercase here or remove it at all to be in line
> with other jobs. But otherwise seems fine to me:
>
> Acked-by: Ilya Maximets <[email protected]>
Thanks Ilya (and Simon), applied the change on commit!
//Eelco
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
